mobile
mobile copied to clipboard
[PM-4794] Trim TOTP key
Type of change
- [x] Bug fix
- [ ] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [ ] Other
Objective
This PR resolves #2122. When cloning a working entry that has a valid OTP and editing the cloned entry by putting a space at the beginning of otpauth:// a wrong OTP is generated.
Code changes
- TotpService.cs: The expected behaviour would be to still have a valid otp and therefore the whitespaces are trimmed. Thanks to @FlorianLang06
Before you submit
- Please check for formatting errors (
dotnet format --verify-no-changes
) (required) - Please add unit tests where it makes sense to do so (encouraged but not required)
- If this change requires a documentation update - notify the documentation team
- If this change has particular deployment requirements - notify the DevOps team
Thank you for your contribution! We've added this to our internal Community PR board for review. ID: PM-4794
Checkmarx One – Scan Summary & Details – 78145036-f3c2-49a7-ad60-703aa5bdcbf2
No New Or Fixed Issues Found
Thank you for your contribution! We've added this to our internal Community PR board for review. ID: PM-4794
Checkmarx One – Scan Summary & Details – 78145036-f3c2-49a7-ad60-703aa5bdcbf2