Skip to content

Conversation

@adrianhoelzl-sap
Copy link
Contributor

No description provided.

@fhanik
Copy link
Contributor

fhanik commented Oct 3, 2025

Thank you @adrianhoelzl-sap
This introduces breaking changes on our end, so I have to coordinate with @duanemay on how/when we want to merge this.

@adrianhoelzl-sap adrianhoelzl-sap force-pushed the remove-origin-threadlocal branch from fa96e23 to f476992 Compare October 8, 2025 08:30
…anager#getExternalUserAuthorities and all overrides
…ticationManagerTest#populateAttributesStoresCustomAttributesAndRoles
@adrianhoelzl-sap adrianhoelzl-sap force-pushed the remove-origin-threadlocal branch from f476992 to 908306a Compare October 9, 2025 09:59
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Oct 9, 2025
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.

Good to merge

@fhanik fhanik merged commit 7e671a0 into develop Oct 9, 2025
33 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Oct 9, 2025
@fhanik fhanik deleted the remove-origin-threadlocal branch October 9, 2025 15:24
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.

Remove ExternalLoginAuthenticationManager.origin

3 participants