Skip to content

fix(deps): go-jose 4.1.6 (GHSA-mgcc-cww5-fp2q) and v0.1.30 - #201

Merged
ndbroadbent merged 2 commits into
mainfrom
nathan/go-jose-4.1.6
Oct 30, 2026
Merged

ndbroadbent merged 2 commits into
mainfrom
nathan/go-jose-4.1.6

Conversation

@ndbroadbent

@ndbroadbent ndbroadbent commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Upgrades github.com/go-jose/go-jose/v4 from 4.1.5 to 4.1.6 for GHSA-mgcc-cww5-fp2q (CVE-2026-116587): parsing a JWK with a very large RSA key could exhaust the CPU. 4.1.6 caps the RSA modulus it accepts at 4096 bits.

The nightly Security Scan's Trivy check of the published image reports it as HIGH, so the release image needs the fix.

  • The gateway includes go-jose through go-oidc. It only parses JWKs and ID tokens it fetches from the configured OIDC provider over TLS (the provider's JWKS and the token endpoint's response to the code exchange), never one a client sends, so no request reaches the affected code.
  • The CLI includes it through go-spiffe, part of the Sigstore verification behind rack-gateway update, which reads trusted roots and attestation bundles.

Also bumps the version to v0.1.30 so the fix can be released. Supersedes #200 (the same go.mod change from Dependabot).

Summary by CodeRabbit

  • Chores
    • Updated the application release to version 0.1.30.
    • Updated the gateway and admin services to use the matching release.

…a JWK

GHSA-mgcc-cww5-fp2q (CVE-2026-116587): parsing a JWK with a very large RSA key could exhaust the CPU. The gateway
includes go-jose through go-oidc (the ID token check), the CLI through go-spiffe (Sigstore verification).
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 9462f3d8-0e9f-4499-acd5-121a44bd83ef
Commits

Reviewing files that changed from the base of the PR and between fe78d6c and 80a70b3.

Files ignored due to path filters: 1
  • go.sum is excluded by !**/*.sum
Files selected for processing: 3
  • convox.yml
  • go.mod
  • web/package.json

Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Walkthrough

The web package version and gateway and admin image tags changed to 0.1.30. The indirect go-jose dependency changed to v4.1.6.

Changes

Release update

Layer / File(s) Summary
Release version and dependency updates
web/package.json, convox.yml, go.mod
The web package version and gateway and admin image tags changed to 0.1.30. The indirect github.com/go-jose/go-jose/v4 dependency changed from v4.1.5 to v4.1.6.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 80a70

The release metadata, image tags, and go-jose dependency update are consistent; the change is mergeable.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the go-jose dependency upgrade and the v0.1.30 release update, which match the main changes.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the tags in rows,
The gateway gets a newer coat.
The admin image follows too,
The web version steps up new.
A small dependency joins the ride,
Then hops away with ears held wide.

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

@deepsource-io

deepsource-io Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in fe78d6c...80a70b3 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Oct 30, 2026 4:38a.m. Review ↗
Go Oct 30, 2026 4:38a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@ndbroadbent
ndbroadbent merged commit dcfcc47 into main Oct 30, 2026
12 checks passed
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.

1 participant