Skip to content
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

using a space at the beginning of otpauth:// generate a wrong OTP #6873

Open
1 task done
flooxo opened this issue Nov 12, 2023 · 1 comment · May be fixed by #6874
Open
1 task done

using a space at the beginning of otpauth:// generate a wrong OTP #6873

flooxo opened this issue Nov 12, 2023 · 1 comment · May be fixed by #6874
Labels
browser Browser Extension bug

Comments

@flooxo
Copy link

flooxo commented Nov 12, 2023

Steps To Reproduce

  1. Open Bitwarden
  2. clone a working entry that has a valid OTP
  3. edit the cloned entry putting a space at the beginning of otpauth://... like this:
    otpauth://...
  4. save

Expected Result

bitwarden deals with wrong characters like spaces before otpauth:// and generate a working OTP

Actual Result

a wrong OTP is generated

Screenshots or Videos

No response

Additional Context

the same issue also exists in the mobile version bitwarden/mobile#2122

Operating System

macOS

Operating System Version

No response

Web Browser

Chrome

Browser Version

No response

Build Version

2023.10.2

Issue Tracking Info

  • I understand that work is tracked outside of Github. A PR will be linked to this issue should one be opened to address it, but Bitwarden doesn't use fields like "assigned", "milestone", or "project" to track progress.
@flooxo flooxo added browser Browser Extension bug labels Nov 12, 2023
@flooxo flooxo linked a pull request Nov 12, 2023 that will close this issue
@SergeantConfused
Copy link

Hi @flooxo,

Thank you for your report and for your PR. This GitHub report will remain open for tracking purposes.

All the best,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser Browser Extension bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants