passwordless-server
passwordless-server copied to clipboard
Step up v2
Ticket
- Contributes to PAS-299
Description
This is the v2 of the step up API (really just adding Purpose to sign in). This also adds the authentication configuration endpoints to allow for admin console (or an API user) to set, get and create new step up purposes.
Shape
- Adds purpose to SignInToken
- Adds purpose to SignInVerifyDTO
- Adds two default purposes of SignIn and StepUp
- Adds new table
AuthenticationConfiguration
- Adds new AdminConsole Settings page.
Screenshots
Addition to Settings.cshtml
Edit
Create
Delete
Checklist
I did the following to ensure that my changes were tested thoroughly:
- validated existing sign in worked
- validated new step up flow works
- validated magic links worked
- validated new admin console settings worked
I did the following to ensure that my changes do not introduce security vulnerabilities:
- validated you could not change tenant from blazor pages
- ran existing integration tests to validate nothing broke
Codecov Report
Attention: Patch coverage is 55.60027%
with 662 lines
in your changes missing coverage. Please review.
Project coverage is 34.11%. Comparing base (
2fbab91
) to head (872af36
).
Additional details and impacted files
@@ Coverage Diff @@
## main #530 +/- ##
==========================================
+ Coverage 33.00% 34.11% +1.11%
==========================================
Files 514 536 +22
Lines 26131 27552 +1421
Branches 774 807 +33
==========================================
+ Hits 8625 9400 +775
- Misses 17378 18021 +643
- Partials 128 131 +3
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
fyi: I noticed there is a merge conflict
fyi: I noticed there is a merge conflict
Fixed 👍