Skip to content

fix(auth): apply offline_access to reauth paths#897

Open
michaelneale wants to merge 1 commit into
mainfrom
micn/sep-2207-offline-access-followup
Open

fix(auth): apply offline_access to reauth paths#897
michaelneale wants to merge 1 commit into
mainfrom
micn/sep-2207-offline-access-followup

Conversation

@michaelneale

Copy link
Copy Markdown
Contributor

Summary

Follow-up for #882 / SEP-2207 to ensure offline_access is applied beyond the initial authorization path.

  • Applies add_offline_access_if_supported to step-up re-authorization scopes.
  • Applies it to refresh-token scope requests when existing granted scopes are non-empty.
  • Adds focused tests for both paths.

Testing

  • cargo test -p rmcp --lib transport::auth --features auth,client,transport-streamable-http-client-reqwest

Closes #882.

@github-actions github-actions Bot added T-core Core library changes T-transport Transport layer changes labels Jun 9, 2026
@michaelneale michaelneale marked this pull request as ready for review June 9, 2026 06:25
@michaelneale michaelneale requested a review from a team as a code owner June 9, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify SEP-2207: OIDC refresh token handling (already implemented via #676)

1 participant