-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Upgrade web-auth/webauthn-lib to version 4.7 #18752
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #18752 +/- ##
============================================
+ Coverage 56.46% 56.51% +0.05%
- Complexity 16137 16143 +6
============================================
Files 673 673
Lines 64142 64175 +33
============================================
+ Hits 36215 36267 +52
+ Misses 27927 27908 -19
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@williamdes, What do you think about requiring this lib for 6.0 and then dropping the custom server that I wrote? We already ship this lib for the website package. I can also work on using this lib to replace the code-lts/u2f-php-server for FIDO U2F. |
I am not sure what you mean about website |
The release package distributed on the website |
We can change this like we want for 6.0 :) |
Hi @MauricioFauth. I was thinking about proposing a change related to phpMyAdmin's WebAuthn-implementation. At this moment the rpId is 'hardcoded' the hostname of the current request. That is a sensible default in most situations, however in some situations it would be nice if you can define the rpId yourself via the config (e.g. when you want to share a WebAuthn-credential across multiple hosts on the same domain). I can propose a separate PR for this, but looking at the changes in this PR it's maybe better to incorporate it on this branch as well? |
Thank you for this suggestion. I'll add this. |
No description provided.