Repository navigation
docs: bring the README and guides up to date with v1.26 and Socrate v1.14 - #123
Merged
Merged
Conversation
Service-account token shape (sub app:<id>, scope, aud under AUDIENCE_MODE=dual since Socrate v1.13.0), RequireScopes with application-defined scopes (Socrate v1.14.0+), package table rows for RequireTenantWith, KeyedRateLimiter and problem details, the guide's BFF example on bff.PendingLoginStore, a non-compiling apierror snippet, the CI job names, and a bff doc comment naming SanitizeReturnTo. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GKRxaeYxyDhmt42cehLsGA
2 of 3 tasks
ovander
pushed a commit
that referenced
this pull request
Oct 11, 2026
Moves [Unreleased] under [1.27.0] - 2026-10-11 (#125 ServiceTokenWithScopes, #123 docs), updates the compare links, the README install snippet and the guide's version note. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GKRxaeYxyDhmt42cehLsGA
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.
What and why
The docs had fallen behind the code and Socrate's token contract. Every statement below was checked against
go doc, the conformance fixtures and go-oauth2's CHANGELOG.README.md:apierrorproblem details,ctxutilscopes and audiences,httpware.RequireTenantWith,jwtauthopt-in checks.KeyedRateLimiterandRequireScopes.jwtauthsection: application scopes need Socrate v1.14.0 or later, and it describes what a service-account token carries.insufficient_scope.buildinfoexample showsgo1.27.2; a missing separator and a gofmt misalignment are fixed.docs/CLIENT-INTEGRATION.md:client_credentials) tokens:sub=app:<id>,scope,audunderAUDIENCE_MODE=dualsince Socrate v1.13.0, plusauth_timeandamr.RequireScopes.bff.NewMemoryPendingLoginStoreinstead of a hand-written map, and compiles against the current API.apierrorconstructor list.insufficient_scope.docs/MIGRATING-TO-SOCRATE.md: service-account tokens andRequireScopes.CONTRIBUTING.md: names the CI jobs.bff/pending_login.go: a doc comment named a non-existentSafeRedirect; it'sSanitizeReturnTo. Comment only.Checked and already correct: every package,
conformanceincluded, has a README table row and a reference section; the Go floor and toolchain versions are right everywhere.How it was tested
go build ./...,go vet ./...andgofmtare clean (the only Go change is a comment)go test/ golangci-lint / govulncheck: CI runs them; no code change## [Unreleased]inCHANGELOG.mdCompatibility
This PR and the v1.26.1 release PR (#122) both touch
CHANGELOG.md. Whichever merges second will needmainmerged in, and I'll do that.🤖 Generated with Claude Code
https://claude.ai/code/session_01GKRxaeYxyDhmt42cehLsGA
Generated by Claude Code