bitwarden / passwordless-server

Bitwarden Passwordless.dev infrastructure/backend (API, database, Docker, etc).
https://bitwarden.com/
Other
83 stars 26 forks source link

PAS-560 | Playground crashing w/ new user without 'Hints' field #677

Closed jonashendrickx closed 1 month ago

jonashendrickx commented 1 month ago

Ticket

Description

Hints field is null, causing it to crash on the line where it tries to split the string by comma separated values.

Shape

Screenshots

Checklist

I did the following to ensure that my changes were tested thoroughly:

I did the following to ensure that my changes do not introduce security vulnerabilities:

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 34.81%. Comparing base (62926dc) to head (6c9e370). Report is 6 commits behind head on main.

Files Patch % Lines
...nConsole/Pages/App/Playground/NewAccount.cshtml.cs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #677 +/- ## ======================================= Coverage 34.81% 34.81% ======================================= Files 566 566 Lines 30985 30985 Branches 926 927 +1 ======================================= Hits 10788 10788 Misses 20055 20055 Partials 142 142 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.