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.

Sign up for GitHub

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

Jump to bottom

TOTP not returning correct codes for seeds with length 26 characters #7

Open
petrm opened this issue May 21, 2016 · 2 comments · May be fixed by #12
Open

TOTP not returning correct codes for seeds with length 26 characters #7

petrm opened this issue May 21, 2016 · 2 comments · May be fixed by #12
Labels
bug hacktoberfest

Comments

@petrm
Copy link

petrm commented May 21, 2016

Hello,
I discovered that the code is not returning correct values for this seed: HG3ST7NLZJJRASIQSLQLZH2EDV

Further testing has shown that all seeds in the same length produce an incorrect code. For timestamp 2016-05-21 07:00:37.995 +0200 CEST the value returned is 502621, but it should be 289787.
I am not sure where to start debugging.
@petrm
Copy link
Author

petrm commented May 21, 2016

Ok, found the problem, the code assumes that the base32 string is padded.

@akerl akerl mentioned this issue Oct 3, 2018
Closed
@f100024 f100024 linked a pull request Jun 23, 2023 that will close this issue
Open
@f100024
Copy link

f100024 commented Jun 23, 2023

Please take a look #12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add secret's aligning f100024/go-otp
3 participants
@hgfischer @petrm @f100024