two-factor icon indicating copy to clipboard operation
two-factor copied to clipboard

Undefined index: hook_suffix

Open kkmuffme opened this issue 6 years ago • 3 comments

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

kkmuffme avatar Jul 21 '19 19:07 kkmuffme

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?

kasparsd avatar Jul 22 '19 14:07 kasparsd

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 avatar Jul 30 '19 13:07 kkmuffme

Any plans to fix this?

kkmuffme avatar Sep 30 '20 10:09 kkmuffme

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 avatar Oct 20 '22 14:10 iandunn