Skip to content

Conversation

@fhanik
Copy link
Contributor

@fhanik fhanik commented Sep 30, 2025

we went from

@RequestMapping(value = "/oauth/clients/restricted", method = RequestMethod.POST)

to

@PostMapping("/oauth/clients/restricted")

It breaks here, urls ending with / no longer work properly, whereas they worked before

and it needs to be

@PostMapping({"/oauth/clients/restricted", "/oauth/clients/restricted/"})

The following URLs now support ending with /. The only endpoint that did not support an ending slash prior to the boot migration was /Users

  1. /Codes
  2. /Groups
  3. /Groups/External
  4. /Groups/zones
  5. /Users
  6. /authenticate
  7. /force_password_change
  8. /identity-zones
  9. /identity-zones/{zoneId}/clients
  10. “{zoneId}/clients
  11. /invite_users
  12. /oauth/clients
  13. /oauth/clients/tx
  14. /oauth/clients/tx/delete
  15. /oauth/clients/tx/modify
  16. /oauth/clients/tx/secret
  17. /password_change
  18. /password_resets
  19. /profile

strehle
strehle previously approved these changes Oct 1, 2025
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Oct 1, 2025
@fhanik fhanik force-pushed the pr/backwards-compatible-request-mappings branch from f3fee3d to 971c244 Compare October 1, 2025 16:08
@fhanik fhanik marked this pull request as ready for review October 1, 2025 17:43
Copy link
Member

@duanemay duanemay left a comment

Choose a reason for hiding this comment

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

Thank you

@fhanik fhanik merged commit 7c4a45d into cloudfoundry:develop Oct 2, 2025
27 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Oct 2, 2025
@fhanik fhanik deleted the pr/backwards-compatible-request-mappings branch October 2, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants