Closed quexten closed 23 hours ago
Attention: Patch coverage is 12.50000%
with 7 lines
in your changes missing coverage. Please review.
Project coverage is 29.29%. Comparing base (
ffb99ef
) to head (1189cfa
).
Files | Patch % | Lines |
---|---|---|
apps/web/src/app/auth/sso.component.ts | 12.50% | 7 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One โ Scan Summary & Details โ 36c2eec9-2cc4-4aec-9025-8f2be216772f
๐๏ธ Tracking
https://bitwarden.atlassian.net/jira/software/c/projects/BW/boards/87?selectedIssue=PM-8718
๐ Objective
If the identifier parameter is passed in the URL, the component will auto-submit the form and redirect to login if the sso identifier does not exist, and to the sso provider if it does exist.
Concerns for reviewing: :question: Right now we just catch all errors from the parent component, the sso prevalidation failing being one of them. Are there other errors, where redirecting to /login after the autosubmit flow failed is a problem?
๐ธ Screenshots
(Recording is in the Jira ticket)
โฐ Reminders before review
๐ฆฎ Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or โน๏ธ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or ๐ญ (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or โ ๏ธ (:warning:
) for more significant problems or concerns needing attention:seedling:
) or โป๏ธ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes