Skip to content

chore: scan golang CVEs - #10641

Merged
olblak merged 4 commits into
updatecli:mainfrom
olblak:scan/cves
Sep 29, 2026
Merged

olblak merged 4 commits into
updatecli:mainfrom
olblak:scan/cves

Conversation

@olblak

@olblak olblak commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Configure Updatecli to actively scan for Golang CVEs with a severity higher than high

Test

To test this pull request, you can run the following commands:

updatecli compose diff --only-policy-ids golang_cve_default

Additional Information

Checklist

  • I have updated the documentation via pull request in website repository.
  • I have tested this pull request manually with a custom Updatecli build and it works as expected.

Tradeoff

Potential improvement

Summary by CodeRabbit

  • New Features
    • Added monitoring for high-severity vulnerabilities in Go modules, helping identify dependencies that may need attention.

Signed-off-by: Olblak <me@olblak.com>
@olblak olblak added dependencies Pull requests that update a dependency file chore labels Sep 28, 2026
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: a9e9fd19-4cff-4a10-8124-9a04d886ef8b

📥 Commits

Reviewing files that changed from the base of the PR and between 52de6d7 and c03a27d.

📒 Files selected for processing (2)
  • updatecli-compose.yaml
  • updatecli/updatecli-compose.d/golang/vulnerability.yaml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The default Updatecli policy list now includes a Go vulnerability policy. The policy sets a high minimum vulnerability severity and scans individual Go modules in go.mod.

Changes

Go vulnerability policy

Layer / File(s) Summary
Configure and include the Go vulnerability policy
updatecli/updatecli-compose.d/golang/vulnerability.yaml, updatecli-compose.yaml
Adds a policy for high-severity Go vulnerabilities in individual Go modules and adds it to the default policy list.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to c03a2

The Go vulnerability policy is included and applies the configured high-severity threshold. No identified issue prevents merging after normal checks.

Security Architecture Review

Security architecture risk: 🔵 Low · up to c03a2

The new scan is limited in configuration to Go modules and a high-severity threshold, but the effective filtering and update behavior have not been verified. No security finding is established.

Retained concerns
No architecture-level concerns identified.

Security review details

Security Blast Radius

  • inferred — If the rendered policy produces updates and has write credentials at runtime, the configured SCM destination is the updatecli/updatecli repository on main. No broader repository or tenant exposure is established by the available configuration.

Trust Boundaries and Controls

  • inferred — The remote policy is the unresolved boundary between the configured severity and module selectors and their effective enforcement. The vulnerability:minhigh label is not evidence that filtering occurs.

Resilience and Maintainability Implications

  • inferred — The local YAML establishes neither the remote policy’s write path nor its ordering, idempotency, conflict handling, or recovery behavior during repeated or interrupted applies.

Hardening Proposals

  • proposed — Verify the rendered pinned policy receives and enforces minseverity=high, then inspect its apply path and repeat/failure behavior before relying on the new policy as a security control.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: scanning Golang CVEs. It is concise and specific.
Description check ✅ Passed The description explains the change, includes a relevant test command, and contains all template sections. The issue reference and detailed tradeoff and potential improvement entries are missing, but …
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

olblak and others added 2 commits September 28, 2026 17:59
Signed-off-by: Olblak <me@olblak.com>
@olblak
olblak enabled auto-merge (squash) September 28, 2026 16:05
@olblak olblak closed this Sep 28, 2026
auto-merge was automatically disabled September 28, 2026 18:08

Pull request was closed

@olblak olblak reopened this Sep 28, 2026
@olblak
olblak enabled auto-merge (squash) September 28, 2026 19:52
@olblak
olblak merged commit 8828e6f into updatecli:main Sep 29, 2026
12 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant