Skip to content

Support newer UAA response fields and grant types#1340

Open
jorbaum wants to merge 5 commits intocloudfoundry:5.x.xfrom
jorbaum:support-new-uaa
Open

Support newer UAA response fields and grant types#1340
jorbaum wants to merge 5 commits intocloudfoundry:5.x.xfrom
jorbaum:support-new-uaa

Conversation

@jorbaum
Copy link

@jorbaum jorbaum commented Mar 13, 2026

Different (likely newer?) versions of the open-source UAA (tested against v78.6.0) return fields and values that the client doesn't model, causing deserialization failures.

This PR adds support for:

  • urn:ietf:params:oauth:grant-type:jwt-bearer grant type (RFC 7523)
  • defaultIdentityProvider in identity zone configuration (added to OSS UAA in 2018)
  • defaultIdpName in the /info endpoint response (source)

Integration test changes:

  • Relaxed assertion for UAA "access denied" error message wording
  • Check for login endpoint instead of password reset endpoint which might be disabled for some UAA

AI tools used: Claude Code and GitHub Copilot (Opus 4.6) assisted me during development. I reviewed the result.

@jorbaum jorbaum force-pushed the support-new-uaa branch 2 times, most recently from 5a63907 to 1d3eae3 Compare March 13, 2026 16:56
@Kehrlann Kehrlann self-requested a review March 24, 2026 10:00
Kehrlann
Kehrlann previously approved these changes Mar 24, 2026
Copy link
Contributor

@Kehrlann Kehrlann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please target 5.x.x

Lokowandtg
Lokowandtg previously approved these changes Mar 24, 2026
@jorbaum jorbaum dismissed stale reviews from Lokowandtg and Kehrlann via f2d3f74 March 26, 2026 10:20
@jorbaum jorbaum changed the base branch from main to 5.x.x March 26, 2026 10:20
@jorbaum
Copy link
Author

jorbaum commented Mar 26, 2026

Rebased on 5.x.x and now targeting it.

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.

3 participants