Skip to content

chore(deps): upgrade golang-jwt from v4.5.2 to v5.3.1 - #21258

Merged
janisz merged 2 commits into
masterfrom
golang-jwt
Aug 3, 2026
Merged

janisz merged 2 commits into
masterfrom
golang-jwt

Conversation

@janisz

@janisz janisz commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Upgrade github.com/golang-jwt/jwt from v4.5.2 to v5.3.1.
Updated import paths from /v4 to /v5 across 4 files.

No API changes required - all used APIs (MapClaims, NewWithClaims,
SigningMethodRS256, ParseWithClaims, RegisteredClaims,
ParseRSAPrivateKeyFromPEM, WithoutClaimsValidation, ErrTokenMalformed)
are compatible with v5.

Changes: golang-jwt/jwt@v4.5.2...v5.3.1

Tested with:

  • go test ./central/auth/m2m -run TestIssuerFromRawIDToken
  • go test ./central/auth/m2m -run Test_genericTokenVerifier_VerifyIDToken

Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

Description

change me!

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

change me!

@openshift-ci

openshift-ci Bot commented Jun 18, 2026

Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@rhybrillou

Copy link
Copy Markdown
Contributor

go mod tidy ?

Base automatically changed from RoaringBitmap to master June 19, 2026 14:01
@janisz
janisz marked this pull request as ready for review July 3, 2026 08:41
@janisz
janisz requested review from a team as code owners July 3, 2026 08:41
@janisz
janisz requested a review from rhybrillou July 3, 2026 08:41
@janisz janisz added the auto-retest PRs with this label will be automatically retested if prow checks fails label Jul 3, 2026
@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Changes

Go dependency cache mtime stabilization

Layer / File(s) Summary
Configurable Go cache mtime handling
.github/actions/cache-go-dependencies/action.yaml
Adds mtime-mode with git-log and blob strategies, stabilizes mtimes for the Go test cache and module index, and keeps directory mtimes fixed.

JWT test token encoding

Layer / File(s) Summary
Encoded JWT test signature
central/jwt/jwt_test.go
Updates the JWT test helper to Base64URL-encode the fake signature segment.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: dependabot[bot]

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR includes a brief summary and test commands, but most required template sections remain as placeholders or unchecked. Fill out the Description, User-facing documentation, Testing and quality, Automated testing, and How I validated my change sections, or explicitly mark items not needed.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title accurately describes the main change: upgrading golang-jwt from v4.5.2 to v5.3.1.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch golang-jwt

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

@janisz janisz added ci-seed-cache Enable cache saving on PR runs (for seeding) ci-save-cache Enable Go cache saves on this PR branch for testing labels Jul 10, 2026
@janisz
janisz changed the base branch from reproduce/jwt-roxvet-stale-cache-index to master July 29, 2026 10:18
@janisz
janisz requested review from a team and rhacs-bot as code owners July 29, 2026 10:18
@rhacs-bot
rhacs-bot requested a review from a team July 29, 2026 10:18
janisz and others added 2 commits July 30, 2026 14:52
Upgrade github.com/golang-jwt/jwt from v4.5.2 to v5.3.1.
Updated import paths from /v4 to /v5 across 4 files.

No API changes required - all used APIs (MapClaims, NewWithClaims,
SigningMethodRS256, ParseWithClaims, RegisteredClaims,
ParseRSAPrivateKeyFromPEM, WithoutClaimsValidation, ErrTokenMalformed)
are compatible with v5.

Changes: golang-jwt/jwt@v4.5.2...v5.3.1

Tested with:
- go test ./central/auth/m2m -run TestIssuerFromRawIDToken
- go test ./central/auth/m2m -run Test_genericTokenVerifier_VerifyIDToken

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
golang-jwt/v5 validates token structure before extracting claims,
rejecting the literal "signature" string as invalid base64.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@janisz

janisz commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@stackrox stackrox deleted a comment from rhacs-bot Jul 31, 2026
@stackrox stackrox deleted a comment from rhacs-bot Jul 31, 2026
@stackrox stackrox deleted a comment from rhacs-bot Jul 31, 2026
@stackrox stackrox deleted a comment from rhacs-bot Jul 31, 2026
@janisz

janisz commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@openshift-ci

openshift-ci Bot commented Jul 31, 2026 •

Copy link
Copy Markdown

@janisz: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ocp-4-21-qa-e2e-tests c09f1d0 link false /test ocp-4-21-qa-e2e-tests
ci/prow/ocp-4-21-nongroovy-e2e-tests c09f1d0 link false /test ocp-4-21-nongroovy-e2e-tests
ci/prow/ocp-4-21-operator-e2e-tests c09f1d0 link false /test ocp-4-21-operator-e2e-tests

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@rhacs-bot

Copy link
Copy Markdown
Contributor

/retest

@janisz

janisz commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

/konflux-retest operator-bundle-on-push

@janisz
janisz merged commit ae72046 into master Aug 3, 2026
202 of 244 checks passed
@janisz
janisz deleted the golang-jwt branch August 3, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review area/central area/ci area/helm area/operator area/scanner area/sensor area/ui auto-retest PRs with this label will be automatically retested if prow checks fails ci-save-cache Enable Go cache saves on this PR branch for testing ci-seed-cache Enable cache saving on PR runs (for seeding) coderabbit-review konflux-build Run Konflux in PR. Push commit to trigger it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants