Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: panic when image pull secret cannot be parsed #764

Merged
merged 1 commit into from
Oct 22, 2021
Merged

fix: panic when image pull secret cannot be parsed #764

merged 1 commit into from
Oct 22, 2021

Conversation

danielpacak
Copy link
Contributor

Indicate which secret cannot be parsed and why:

error: reading .dockerconfigjson field of "default/regcred" secret: expected
username and password concatenated with a colon (:)

Resolves: #751

Signed-off-by: Daniel Pacak [email protected]

Indicate which secret cannot be parsed and why:

    error: reading .dockerconfigjson field of "default/regcred" secret:
    expected username and password concatenated with a colon (:)

Resolves: #751

Signed-off-by: Daniel Pacak <[email protected]>
@codecov
Copy link

codecov bot commented Oct 20, 2021

Codecov Report

Merging #764 (1118d5a) into main (74c6501) will increase coverage by 0.06%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #764      +/-   ##
==========================================
+ Coverage   65.53%   65.59%   +0.06%     
==========================================
  Files          58       58              
  Lines        6305     6308       +3     
==========================================
+ Hits         4132     4138       +6     
+ Misses       1759     1757       -2     
+ Partials      414      413       -1     
Impacted Files Coverage Δ
pkg/kube/secrets.go 60.00% <0.00%> (ø)
pkg/docker/config.go 71.21% <100.00%> (+6.13%) ⬆️
pkg/operator/controller/ciskubebenchreport.go 57.69% <0.00%> (-2.14%) ⬇️
pkg/operator/controller/configauditreport.go 67.26% <0.00%> (+1.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74c6501...1118d5a. Read the comment docs.

@danielpacak danielpacak merged commit 8944e54 into aquasecurity:main Oct 22, 2021
@danielpacak danielpacak deleted the fix/issue_751 branch October 22, 2021 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Starboard operator crashes with panic: runtime error: index out of range [1] with length 1
2 participants