← 返回首页
Support newer UAA response fields and grant types by jorbaum · Pull Request #1340 · cloudfoundry/cf-java-client · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Resetting focus

Support newer UAA response fields and grant types#1340

Merged
Kehrlann merged 5 commits into
cloudfoundry:5.x.xfrom
jorbaum:support-new-uaa
Mar 27, 2026
Merged

Support newer UAA response fields and grant types#1340
Kehrlann merged 5 commits into
cloudfoundry:5.x.xfrom
jorbaum:support-new-uaa

Conversation

jorbaum commented Mar 13, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

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 force-pushed the support-new-uaa branch 2 times, most recently from 5a63907 to 1d3eae3 Compare March 13, 2026 16:56
Kehrlann self-requested a review March 24, 2026 10:00
Kehrlann previously approved these changes Mar 24, 2026

Kehrlann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

LGTM, please target 5.x.x

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

jorbaum commented Mar 26, 2026

Copy link
Copy Markdown
Contributor Author

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

Hide details View details Kehrlann merged commit d100933 into cloudfoundry:5.x.x Mar 27, 2026
1 check passed
jorbaum deleted the support-new-uaa branch March 27, 2026 10:51
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Footer

© 2026 GitHub, Inc.