[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