[wp-trac] [WordPress Trac] #51941: Unique names for Application Password

WordPress Trac noreply at wordpress.org
Fri Jan 22 21:50:25 UTC 2021


#51941: Unique names for Application Password
-------------------------------------------------+-------------------------
 Reporter:  Boniu91                              |       Owner:  boniu91
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.7
Component:  Application Passwords                |     Version:  5.6
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing has-unit-    |     Focuses:
  tests                                          |  administration
-------------------------------------------------+-------------------------

Comment (by xkon):

 @TimothyBlynJacobs yes we do use esc_html on outputs, but we'll have to
 adjust the JS to avoid showing possible returned `<` and other
 characters.

 @hellofromTonya would you like to update your PR for the changes mentioned
 in comments [comment:14] & [comment:15]? I see that you already pushed a
 trim() also in the PR but if we pass the Name via `sanitize_text_field` it
 won't be needed afaic.

 If it's easier we can split the extra changes mentioned on a new task and
 I can also work on them tomorrow, to commit this "as is" since tests are
 OK :).

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


More information about the wp-trac mailing list