Skip to content

Conversation

reshnm
Copy link
Contributor

@reshnm reshnm commented Sep 29, 2025

What this PR does / why we need it:

This PR fixes the following issues:

  • Handle MCPs with an OIDC config of nil
  • Condition Types are not allowed to contain :. Change : to _ for the provider prefix.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Correctly handle MCPs without an OIDC config
Change `:` to `_` for the access secret provider prefix.

@reshnm reshnm requested a review from Diaphteiros September 29, 2025 11:46
@reshnm reshnm changed the title fix: access request fix: access request handling for MCP V2 Sep 29, 2025
@reshnm reshnm merged commit f4e38be into main Sep 29, 2025
6 checks passed
@reshnm reshnm deleted the fix/access-request branch September 29, 2025 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants