[wp-trac] [WordPress Trac] #32508: Action Hooks for empty username and empty password

WordPress Trac noreply at wordpress.org
Wed May 27 20:24:51 UTC 2015


#32508: Action Hooks for empty username and empty password
-------------------------+------------------------------
 Reporter:  jwarren      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Plugins      |     Version:  trunk
 Severity:  normal       |  Resolution:
 Keywords:               |     Focuses:
-------------------------+------------------------------

Comment (by jwarren):

 Replying to [comment:1 MikeHansenMe]:
 > Instead of adding new hooks is there a reason we exclude those from
 triggering 'wp_login_failed'? If not we could just pass the additional
 param into that hook. [attachment: 32508.2.diff] is an example and still
 needs some docs.

 That was actually my initial idea and would definitely be a cleaner
 option. But I'm concerned about breaking current implementations. If
 people don't expect their action to fire on `wp_login_failed` then it
 could go awry. We also don't have a `$username` to pass if we're failing
 due to `empty_username`. We could end up with white screen of death PHP
 errors in code that expects it.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/32508#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list