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

add explicit info for ext-gmp or ext-bcmath and earlier exception handling if not available #3

Open
jpmschuler opened this issue Feb 22, 2023 · 0 comments · May be fixed by jpmschuler/mfa_webauthn#1

Comments

@jpmschuler
Copy link

jpmschuler commented Feb 22, 2023

I ran into installing the extension, configuring my auth token, but not being able to login with "Verification error". I only found in the logs that ext-gmp or ext-bcmath are necessary. Installing ext-gmp resolved any issues and made the extension wonderfully zero-con.

I propose to

  • add an info in the README about this
  • add a composer.json suggests for ext-gmp perhaps
  • check for ext-gmp or ext-bcmath before/during enabling the auth mechanism for a user (so early and where visible messages and exceptions are possible) instead of during first auth where only general logging is possible

(can supply PR if I circle back to this as soon as time comes up)

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 a pull request may close this issue.

1 participant