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

Undefined index: hook_suffix #298

Closed
kkmuffme opened this issue Jul 21, 2019 · 4 comments
Closed

Undefined index: hook_suffix #298

kkmuffme opened this issue Jul 21, 2019 · 4 comments
Labels

Comments

@kkmuffme
Copy link
Contributor

kkmuffme commented Jul 21, 2019

Version 0.4.7

when registering a yubi key in profile I get:

PHP Error: Undefined index: hook_suffix
\wp-admin\includes\class-wp-screen.php on line 229
Error URL: https://myurl.com/wp-admin/admin-ajax.php
Trace:
#1 \wp-admin\includes\template.php(1988): WP_Screen::get(NULL)
#2 \wp-admin\includes\class-wp-list-table.php(143): convert_to_screen(NULL)
#3 \wp-content\plugins\two-factor\providers\class.two-factor-fido-u2f-admin.php(313): WP_List_Table->__construct()
#4 \wp-includes\plugin.php(463): Two_Factor_FIDO_U2F_Admin::wp_ajax_inline_save()
#5 \wp-admin\admin-ajax.php(99): do_action(wp_ajax_inline-...)
@kasparsd
Copy link
Collaborator

Thanks for reporting the issue @kkmuffme!

Looks like it is happening here:

https://github.com/georgestephanis/two-factor/blob/ab8d6510a17606b200d1e44a603c13b5014bac52/providers/class.two-factor-fido-u2f-admin.php#L313

Is that the first time you're registering the key? Are there any other keys current associated with the account?

@kkmuffme
Copy link
Contributor Author

kkmuffme commented Jul 30, 2019

No the key was registered before. But I deleted the key (by clicking on "Delete")
Then later again registered the key again.

Yes, there were other keys associated before, but I also deleted them before adding the current one.

@kkmuffme
Copy link
Contributor Author

Any plans to fix this?

@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.

@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