test(e2e): pin provider v0.12.1 release cut before tagging - #31
Merged
Conversation
Pre-tag E2E for the auth.provider patch release, in the order the release rule requires: cut merged -> umbrella E2E green -> tag. PROVIDER_REV moves from 19ce4c7f (v0.12.0) to 84609f33f0fd704c260735fd918413f0ceebfe13 (release: v0.12.1, #519 + #523). PROXY_REV / VERIFIER_REV / PROVIN_REV are unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
The change is a straightforward, isolated component revision pin aligned with the stated pre-tag E2E intent.
Pull request overview
Updates the E2E harness to run against the provider commit intended to become v0.12.1, so the umbrella integration checks can be executed in the required order (cut merge → E2E green → tag) before tagging the release.
Changes:
- Bump
PROVIDER_REVfrom the previousv0.12.0commit to84609f33f0fd704c260735fd918413f0ceebfe13(pre-tag cut forv0.12.1). - Leave
PROXY_REV,VERIFIER_REV, andPROVIN_REVunchanged.
File summaries
| File | Description |
|---|---|
| Makefile | Pins PROVIDER_REV to the v0.12.1 cut commit to run the umbrella E2E suite against the release candidate. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Pre-tag E2E for one patch release, in the order the release rule requires: cut merged → umbrella E2E green → tag.
19ce4c7f(v0.12.0)84609f33(#523 on top of #519)v0.12.1The patch contains only the jwt-bearer
audfix (#518): the grant now mintsaudas the client'sallowedAudiences[0](falling back to the client id) like the session and device grants, honours a policygrantedAudiencefail-closed againstallowedAudiences, and refuses a policy audience for an unauthenticated caller. Fixed-only, no BREAKING, exported surface unchanged. Nothing in this harness or in Provin exercises jwt-bearer, so this run is the regression check for the rest of the stack at the cut commit.PROXY_REV,VERIFIER_REVandPROVIN_REVare unchanged. The tag is pushed only after this run is green.🤖 Generated with Claude Code