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

Deleting physical security key fails after renaming #431

Closed
Devvypaws opened this issue Feb 26, 2022 · 1 comment
Closed

Deleting physical security key fails after renaming #431

Devvypaws opened this issue Feb 26, 2022 · 1 comment
Labels

Comments

@Devvypaws
Copy link

Hello! I came across the following issue involving an error with the physical security key interface under the Profile page:

How to Reproduce Issue:

  1. Register new security key
  2. Rename the security key
  3. Delete the now-renamed security key

Expected Results
Page reloads with the security key removed

Actual Results:
Page returns an HTTP 400 Bad Request, page content is just 0, and the security key is still registered

Environment:

  • WordPress 5.9.1
  • PHP 8.1.3
  • MySQL 8.0.26
  • Two Factor 0.7.1

Additional Notes:
This issue does not occur if you rename the security key, refresh the page, and then delete it.

@iandunn
Copy link
Member

iandunn commented Oct 20, 2022

U2F is deprecated and no longer works in Chrome, so the provider is being removed in #439 . Given that, there's probably no reason to keep this open anymore.

A WebAuthn provider is being added in #427 , which currently uses a different table for listing keys. If this bug reappears once that's released, we could reopen it.

@iandunn iandunn closed this as completed Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants