Skip to content

Aut 5230/add state usage in amc oauth flow#8115

Draft
BeckaL wants to merge 4 commits intomainfrom
AUT-5230/add-state-usage-in-amc-oauth-flow
Draft

Aut 5230/add state usage in amc oauth flow#8115
BeckaL wants to merge 4 commits intomainfrom
AUT-5230/add-state-usage-in-amc-oauth-flow

Conversation

@BeckaL
Copy link
Copy Markdown
Contributor

@BeckaL BeckaL commented Apr 2, 2026

authentication-api PR template picker

Tip

Please go to the Preview tab and select the appropriate sub-template

Templates

BeckaL added 4 commits April 2, 2026 14:19
Will be used to store and retrieve amc state. This is not yet used anywhere, but will be used in the amc oauth flow. The tables have already been created in infrastructure.
This is a preparatory step ahead of storing the state in the authorize handler, so that it can be verified in the callback from amc
A subsequent commit will read the parameter from the request in the callback and check that it matches the client session here
Here we ensure that the state parameter on the original request matches the one in the callback.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

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.

1 participant