Repository navigation
build(deps): upgrade go to 1.27 and update dependencies - #325
Open
northtyphoon wants to merge 1 commit into
Open
northtyphoon wants to merge 1 commit into
northtyphoon wants to merge 1 commit into
Conversation
northtyphoon
requested review from
NiazFK,
gokarnm,
priteshbandi,
rgnote,
shizhMSFT,
toddysm,
vaninrao10 and
yizha1
as code owners
October 25, 2026 06:45
There was a problem hiding this comment.
🟢 Approval recommended
The JWT migration matches the updated API, toolchain settings are aligned, and no blocking issues were identified.
No open issues 🎉
What changed in this PR
This PR updates the module and CI toolchains to Go 1.27 and upgrades dependencies, including the JWS integration changes required by JWT v5.
Changes:
- Require Go 1.27.3 and align build and CodeQL workflows on Go 1.27.
- Upgrade CBOR, JWT, and x/crypto dependencies and refresh checksums.
- Adapt remote signing to JWT v5’s raw-signature API and update verification-error assertions.
| File | Description |
|---|---|
signature/jws/types.go |
Switch JWT types to v5. |
signature/jws/jwt.go |
Use JWT v5 and return raw signature bytes. |
signature/jws/jwt_test.go |
Adapt the verification test to the byte-slice API. |
signature/jws/envelope.go |
Switch envelope processing to JWT v5. |
signature/jws/envelope_test.go |
Update expected JWT verification errors. |
go.sum |
Refresh upgraded dependency checksums. |
go.mod |
Raise the Go requirement and upgrade dependencies. |
.github/workflows/reusable-codeql.yml |
Run CodeQL with Go 1.27. |
.github/workflows/reusable-build.yml |
Run builds with Go 1.27. |
Review effort: Balanced
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: ffaffde9-b3bf-47ff-b8fb-fc774e78a159 Signed-off-by: Bin Du <bindu@microsoft.com>
northtyphoon
force-pushed
the
northtyphoon/upgrade-go-1.27-dependencies
branch
from
October 25, 2026 07:17
b3a3ca0 to
ea81060
Compare
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.
Objective
Upgrade the project to Go 1.27 and the latest direct dependency releases. Keep the module requirements and CI toolchain aligned, including the compatibility changes required by JWT v5.
Changes
Validation
go test ./...: all packages pass except the existing live timestamp test, which expects an outdated DigiCert certificate subject. The same failure reproduces with the original dependencies.go test ./internal/timestamp -skip '^TestTimestamp$/^Timestamping_certificate_revoked$': passes.go vet ./...: reports existing lock-copy warnings insignature/envelope_test.go; these also reproduce with the original dependencies.go mod verify: passes.