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

Account or Service field containing colon breaks the Test feature in the advanced form #45

Closed
Bubka opened this issue Sep 9, 2021 · 2 comments
Assignees
Labels
bug Something isn't working fixed The issue is fixed (in a coming release)

Comments

@Bubka
Copy link
Owner

Bubka commented Sep 9, 2021

Describe the bug
The Test feature returns an error in the advanced form when the Account or Service field contains a colon.

To Reproduce
Steps to reproduce the behavior:

  1. Go to /account/create
  2. Field the form with Account or Service with a colon
  3. Click on Test
  4. See error

Expected behavior
An OTP password should be displayed

@Bubka Bubka added the bug Something isn't working label Sep 9, 2021
@Bubka Bubka changed the title Account or Service with colon breaks the Test feature in advanced form Account or Service field containing colon breaks the Test feature in the advanced form Sep 9, 2021
@Bubka Bubka added the fixed The issue is fixed (in a coming release) label Sep 9, 2021
@Bubka Bubka self-assigned this Sep 9, 2021
@CoReYeDe
Copy link

CoReYeDe commented Apr 11, 2022

Bug still present..
If an Microsoft-2FA-Account is added, it contains the organization before the accountname, seperated with a colon..

MY ORGANISATION:ACCOUNT.NAME@DOMAIN.COM

cant add it with this error.

image

@Bubka
Copy link
Owner Author

Bubka commented Apr 11, 2022

Thanks for your feedback.
For now the bug has been fixed in the dev branch only, the master branch will be fixed when the v3 of 2FAuth will be released (coming soon, i'm working on it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed The issue is fixed (in a coming release)
Projects
None yet
Development

No branches or pull requests

2 participants