Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ato 975/send current credential strength claim to backed #2164

Merged

Conversation

isaac-GDS
Copy link
Contributor

@isaac-GDS isaac-GDS commented Oct 10, 2024

##What

Take the currentCredentialStrength from the authorise request and pass it on to the start handler so it can be set in the auth session. The currentCredentialStrength might not exist in the authorise request and therefore wont pass the field through to the start handler.

@isaac-GDS isaac-GDS force-pushed the ATO-975/send-current-credential-strength-claim-to-backed branch 2 times, most recently from 8ade2e1 to f7c34ea Compare October 22, 2024 13:38
@isaac-GDS isaac-GDS marked this pull request as ready for review October 29, 2024 10:21
@isaac-GDS isaac-GDS requested review from a team as code owners October 29, 2024 10:21
 - Orchestration are sending the currentCredentialStrength claim
   in the authorize response
 - update the type and test to show this
 - We need to pass the claim `currentCredentialStrength` to the backend
   and store in the auth session. Passing it from the authorize response
   to the start handler to be stored.
@isaac-GDS isaac-GDS force-pushed the ATO-975/send-current-credential-strength-claim-to-backed branch from f7c34ea to 806b669 Compare October 30, 2024 10:30
Copy link

sonarcloud bot commented Oct 30, 2024

@isaac-GDS isaac-GDS merged commit 2ee0225 into main Oct 30, 2024
8 checks passed
@isaac-GDS isaac-GDS deleted the ATO-975/send-current-credential-strength-claim-to-backed branch October 30, 2024 10:43
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.

2 participants