Skip to content

Update security scan config (and .go-version)#771

Merged
tvoran merged 3 commits intomainfrom
VAULT-37239/fix-security-scan
Jun 20, 2025
Merged

Update security scan config (and .go-version)#771
tvoran merged 3 commits intomainfrom
VAULT-37239/fix-security-scan

Conversation

@tvoran
Copy link
Member

@tvoran tvoran commented Jun 19, 2025

Updates .release/security-scan.hcl to be more in line with Vault's, and relies on the OSV vulnerability database for container scans instead of just alpine_secdb/alpine_security. Updates .go-version in line with the 1.24 bump from the last dependabot update.

Scanning results:

❯ make build GOARCH=arm64 && mkdir -p dist/linux/arm64 && cp dist/vault-k8s dist/linux/arm64/ && \
    docker build --build-arg VERSION=0.0.0-dev --no-cache --target default -t docker.io/hashicorp/vault-k8s:0.0.0-dev .

❯ SECURITY_SCANNER_CONFIG_FILE=./.release/security-scan.hcl scan binary dist/vault-k8s                                           
✓ Scanned file:{path:"dist/vault-k8s"} in 5.2s - no results found

❯ SECURITY_SCANNER_CONFIG_FILE=./.release/security-scan.hcl scan container --local-daemon docker.io/hashicorp/vault-k8s:0.0.0-dev
✓ Scanned local_daemon:true registry:{host:"index.docker.io" repository:"hashicorp/vault-k8s" tag:"0.0.0-dev"} in 31.3s - no results found

tvoran added 3 commits June 18, 2025 11:25
Updates security-scan.hcl to be more in line with Vault's, and relies
on the OSV vulnerability database for container scans instead of just
alpine_secdb/alpine_security.
@tvoran tvoran requested a review from a team as a code owner June 19, 2025 06:06
@tvoran
Copy link
Member Author

tvoran commented Jun 20, 2025

Thanks!

@tvoran tvoran merged commit 3c07231 into main Jun 20, 2025
11 checks passed
@tvoran tvoran deleted the VAULT-37239/fix-security-scan branch June 20, 2025 21:37
@tvoran tvoran added this to the 1.7.0 milestone Jun 24, 2025
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.

2 participants