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

Validate a TOTP code before saving it #1055

Merged
merged 25 commits into from
Sep 11, 2024

Conversation

kekonn
Copy link
Contributor

@kekonn kekonn commented Sep 10, 2024

In order to do this, I had to remove the very old and unmaintainted otplib and replace it with the well-maintained otpauth.

I kept the parameters the same, but if there is something only slightly off between the two libraries, this could cause a major headache. I am unsure on how to test this though, or if we can provide some sort of migration for this.

The reason for the upgrade is two-fold:

  1. As I said, the original library was unmaintained
  2. It's verification mechanism relied on a Buffer class that, as far as I can tell, can't work without some require statements that are missing from the library's source files.

kekonn and others added 23 commits September 5, 2024 18:29
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 7.17.0 to 7.18.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v7.18.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
…eb/typescript-eslint/parser-7.18.0

Bump @typescript-eslint/parser from 7.17.0 to 7.18.0 in /warpgate-web
@kekonn
Copy link
Contributor Author

kekonn commented Sep 10, 2024

This would close #977

@Eugeny
Copy link
Member

Eugeny commented Sep 10, 2024

Tested and everything looks good, I've added a few fixed on top 👍

I'll merge it unless you want to add anything else

@kekonn kekonn marked this pull request as ready for review September 11, 2024 06:09
@kekonn
Copy link
Contributor Author

kekonn commented Sep 11, 2024

Fine by me.

@Eugeny Eugeny merged commit 3c3b843 into warp-tech:main Sep 11, 2024
5 of 7 checks passed
@kekonn kekonn deleted the feat/totp-validation branch September 11, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants