Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
e7ab6f3
Add Docker curation support
basel1322 Dec 10, 2025
a7b6843
Improved Docker curation to use docker pull and add Accept headers fo…
basel1322 Dec 10, 2025
a814b8d
Simplify Docker curation code
basel1322 Dec 10, 2025
57396a7
Merge branch 'dev' into docker-curation-supprot
basel1322 Dec 14, 2025
ac16c65
Fixed unit test for Docker curation
basel1322 Dec 14, 2025
3171ad9
improved the code
basel1322 Dec 16, 2025
ecd4a99
fixed unit test commit
basel1322 Dec 16, 2025
4c0adc9
added get version for docker
basel1322 Dec 16, 2025
958ee03
Add Docker curation audit integration test
basel1322 Dec 18, 2025
b22d8cf
Add Docker login in test and fix panic on parse error
basel1322 Dec 18, 2025
d1a5492
Use same test setup pattern as Docker scan tests
basel1322 Dec 18, 2025
cb6f4e5
Use anonymous Docker access for curation test
basel1322 Dec 18, 2025
2e5ad3d
Skip Docker curation test on macOS CI
basel1322 Dec 18, 2025
59a3a7e
Update commands/curation/curationaudit.go
basel1322 Dec 18, 2025
490440d
Update curationaudit.go
basel1322 Dec 18, 2025
06c852a
Update commands/curation/curationaudit.go
basel1322 Dec 21, 2025
5337c9c
Refactor Docker image parsing and remove redundant checkDockerSupport
basel1322 Dec 21, 2025
5f568e8
Fix unparam lint issue
basel1322 Dec 21, 2025
97100eb
feat: get arch-specific digest for successful docker pulls using buildx
basel1322 Dec 22, 2025
207e9b1
fix: fixed typo in buildx inspect debug logs
basel1322 Dec 22, 2025
fe4c435
fix: show actual error message for docker buildx imagetools inspect f…
basel1322 Dec 22, 2025
ae9c166
fix: properly return error from docker buildx imagetools inspect
basel1322 Dec 22, 2025
32bc73c
improved the code
basel1322 Dec 29, 2025
33ee5b1
improved text
basel1322 Dec 29, 2025
4487529
improved the code
basel1322 Dec 30, 2025
8e02730
Clarify Docker image name flag description
basel1322 Dec 30, 2025
3370714
Fix formatting of error message for repository check
basel1322 Dec 30, 2025
ff53b41
Merge branch 'dev' into docker-curation-supprot
basel1322 Dec 30, 2025
60561ec
Fix linter errors: return proper error, lowercase error strings
basel1322 Dec 30, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: show actual error message for docker buildx imagetools inspect f…
…ailures
  • Loading branch information
basel1322 committed Dec 22, 2025
commit fe4c43559c0e54d2444cac08f223c69a6d67fc7a
6 changes: 3 additions & 3 deletions sca/bom/buildinfo/technologies/docker/docker.go
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@
inspectCmd := exec.Command("docker", "inspect", fullImageName, "--format", "{{.Os}} {{.Architecture}}")
inspectOutput, inspectErr := inspectCmd.CombinedOutput()
if inspectErr != nil {
log.Debug(fmt.Sprintf("docker inspect failed: %v", inspectErr))
log.Error(fmt.Sprintf("docker inspect failed: %v", inspectErr))
return "", nil
}
parts := strings.Fields(strings.TrimSpace(string(inspectOutput)))
Expand All @@ -154,13 +154,13 @@
buildxCmd := exec.Command("docker", "buildx", "imagetools", "inspect", fullImageName, "--raw")
buildxOutput, buildxErr := buildxCmd.CombinedOutput()
if buildxErr != nil {
log.Debug(fmt.Sprintf("docker buildx imagetools inspect failed: %v", buildxErr))
log.Error(fmt.Sprintf("docker buildx imagetools inspect failed: %s", strings.TrimSpace(string(buildxOutput))))
return "", nil

Check failure on line 158 in sca/bom/buildinfo/technologies/docker/docker.go

View workflow job for this annotation

GitHub Actions / Static-Check

error is not nil (line 155) but it returns nil (nilerr)
}

var manifest dockerManifestList
if err := json.Unmarshal(buildxOutput, &manifest); err != nil {
log.Debug(fmt.Sprintf("Failed to parse manifest JSON: %v", err))
log.Error(fmt.Sprintf("Failed to parse manifest JSON: %v", err))
return "", nil
}

Expand Down
Loading