Repository navigation
fix(snyk): remediate high-and-above vulnerabilities on main - #7431
Open
prodsec-github-automation wants to merge 1 commit into
Open
prodsec-github-automation wants to merge 1 commit into
prodsec-github-automation wants to merge 1 commit into
Conversation
…ource Applied by snyk fix --agentic via the Snyk ProdSec CircleCI orb, from fix/CLI-1843_legacycli-placeholder-token at 4fb5b5e. These changes are generated. Review them before merging.
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
PR Reviewer Guide 🔍
|
Contributor
|
andrewrobinsonhodges-snyk
added a commit
that referenced
this pull request
Oct 27, 2026
Unblocks the Snyk Open Source gate in the code-analysis CI job, which flags SNYK-GOLANG-GITHUBCOMKLAUSPOSTCOMPRESSFSE-21755459, SNYK-GOLANG-GITHUBCOMKLAUSPOSTCOMPRESSHUFF0-21755460 and SNYK-GOLANG-GITHUBCOMGOJOSEGOJOSEV4-21755478 in both Go modules. Same remediation as #7431 on main. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
4 of 9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Snyk agentic fix
The Snyk Open Source scan of
fix/CLI-1843_legacycli-placeholder-tokenreported vulnerabilities at or above high severity. This branch was produced bysnyk fix --agenticworking on those.The fix itself was applied to
main, not tofix/CLI-1843_legacycli-placeholder-token, so this pull request stands on its own rather than stacking on the branch that triggered it. A vulnerability that exists only onfix/CLI-1843_legacycli-placeholder-tokenis therefore not addressed here.Whether that scan also failed the build depends on the Enhanced Gate, which blocks only once a vulnerability has passed its remediation SLA — so this pull request may exist for a build that is green.
3 of 5 fixed.
Fixed
cliv2-private/go.mod,cliv2/go.modgo-joseintroduces stricter validation that could affect existing applications. Behavioral Change: - v4.1.6: The library now rejects JSON Web Encryption (JWE) messages with mismatched Content Encryption Key (CEK) lengths. Previously, these may have been processed, but now they will result in an error. Recommendation: Verify that any systems generating JWE tokens for consumption by this service are correctly formatting the CEK. This change improves security and spec compliance but could cause issues with incorrectly generated tokens. Source: GitHub Release Notescliv2-private/go.mod,cliv2/go.modcliv2-private/go.mod,cliv2/go.modhuff0decoder, leading to a denial of service. - Optimizations: The releases within this range include performance improvements for thehuff0package. There are no documented breaking API changes. The upgrade is recommended to patch the security vulnerability. Source: Release NotesNot fixed
package.jsonpackage.jsonA row marked Fix available: No has no upgrade path for
snyk fixto take. One marked Yes does, but needed a change the agent would not make unattended — those are the rows to look at first.This is not necessarily a complete fix. The build on this pull request runs the same Open Source scan and quality gate, so its result — not this description — is the verdict on what is left.
Changes
Snyk ProdSec orb · build 685422 · model
claude-sonnet-5-5Note
Medium Risk
Touches crypto/JWE validation (go-jose) and decompression of untrusted data (klauspost/compress) via transitive deps; go-jose v4.1.6 may reject previously accepted malformed JWE tokens.
Overview
Bumps indirect Go dependencies in
cliv2andcliv2-privateto address Snyk high-severity findings. No application source changes—onlygo.mod/go.sum.github.com/go-jose/go-jose/v44.1.4→4.1.6fixes inefficient algorithmic complexity in JWE/JWS handling. v4.1.6 also rejects JWE with mismatched CEK lengths (stricter spec compliance), which could break interaction with peers that emit malformed tokens.github.com/klauspost/compress1.19.2→1.20.2patches integer-overflow / crafted-input issues in zstd and huff0 decoders that could panic (DoS) on malicious compressed data.Reviewers should treat this like any dependency security bump: confirm changelogs and that CLI paths that decompress untrusted input or verify JWE/JWT via transitive stacks behave as expected.
Reviewed by Cursor Bugbot for commit eecbcb0. Bugbot is set up for automated code reviews on this repo. Configure here.