-
Notifications
You must be signed in to change notification settings - Fork 78
Updating SDK to 8266e16 (1.0.0-2742-fc75b90) #2045
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
Conversation
Great job! No new security vulnerabilities introduced in this pull request |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2045 +/- ##
==========================================
- Coverage 85.28% 82.94% -2.35%
==========================================
Files 1690 1987 +297
Lines 144061 160900 +16839
==========================================
+ Hits 122862 133455 +10593
- Misses 21199 27445 +6246 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
await flightRecorder.log("[Auth] Set PIN-protected user key in memory") | ||
} | ||
case .pinEnvelope: | ||
case .masterPasswordUnlock, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎨 Master password unlock should follow the same flow as password unlock; so even though we aren't using it yet, I think we should move this alongside the .password
case above which configures PIN unlock after unlocking the vault.
case .masterPasswordUnlock: | ||
"Master Password Unlock" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⛏️ Can you alphabetize this relative to the other cases?
Updates the SDK from
1e16ec0e5e3f7045a82b747dc9cf5ddf06e13448
to8266e167ae8e80f1bf1e9ab96ee5a4d46162c059
What's Changed
Raw changelog