bitwarden / android

Bitwarden mobile app for Android.
https://bitwarden.com
GNU General Public License v3.0
6.57k stars 823 forks source link

PM-15147 - Design Audit - Master Password Guidance Screen #4383

Open phil-livefront opened 4 days ago

phil-livefront commented 4 days ago

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-15147

📔 Objective

📸 Screenshots

CompleteRegistration MasterPasswordGuidanceScreen

⏰ Reminders before review

🦮 Reviewer guidelines

github-actions[bot] commented 4 days ago

Logo Checkmarx One – Scan Summary & Detailseea1bb35-3c8f-4410-a6e4-8776cffd948c

New Issues

Severity Issue Source File / Package Checkmarx Insight
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/completeregistration/CompleteRegistrationViewModel.kt: 346 Attack Vector
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/completeregistration/CompleteRegistrationViewModel.kt: 346 Attack Vector
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/createaccount/CreateAccountViewModel.kt: 321 Attack Vector
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/createaccount/CreateAccountViewModel.kt: 321 Attack Vector
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/vault/feature/item/VaultItemViewModel.kt: 875 Attack Vector

Fixed Issues

Severity Issue Source File / Package
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/completeregistration/CompleteRegistrationViewModel.kt: 341
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/auth/feature/completeregistration/CompleteRegistrationViewModel.kt: 341
MEDIUM Unpinned Actions Full Length Commit SHA /crowdin-pull.yml: 26
codecov[bot] commented 4 days ago

Codecov Report

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

Project coverage is 89.01%. Comparing base (cce9bef) to head (47c5e86).

Files with missing lines Patch % Lines
...erpasswordguidance/MasterPasswordGuidanceScreen.kt 98.68% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4383 +/- ## ======================================= Coverage 89.01% 89.01% ======================================= Files 451 451 Lines 39129 39142 +13 Branches 5534 5534 ======================================= + Hits 34831 34844 +13 Misses 2370 2370 Partials 1928 1928 ```

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


🚨 Try these New Features: