[wp-trac] [WordPress Trac] #44589: password reset email link faulty in some email clients

WordPress Trac noreply at wordpress.org
Sat Jan 18 02:30:37 UTC 2020


#44589: password reset email link faulty in some email clients
----------------------------------------+-----------------------------
 Reporter:  sproutchris                 |       Owner:  SergeyBiryukov
     Type:  enhancement                 |      Status:  closed
 Priority:  normal                      |   Milestone:  Future Release
Component:  Mail                        |     Version:
 Severity:  normal                      |  Resolution:  fixed
 Keywords:  has-patch needs-unit-tests  |     Focuses:
----------------------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * status:  reviewing => closed
 * resolution:   => fixed


Comment:

 In [changeset:"47086" 47086]:
 {{{
 #!CommitTicketReference repository="" revision="47086"
 Mail: Remove angle brackets from password reset URL in emails sent by
 `retrieve_password()` and `wp_new_user_notification()`.

 The brackets were originally added in [16285] per W3C recommendation in
 https://www.w3.org/Addressing/URL/5.1_Wrappers.html to avoid wrapping the
 URL across multiple lines in plain text in older email clients.

 This doesn't seem like a common issue in modern email clients, and the
 current implementation causes more issues than it solves. Since the URL is
 on a line by itself, it should not require any delimiters.

 The URL in recovery mode email introduced in [44973] doesn't have angle
 brackets, so it's time to retire them in password reset email too if
 they're not used consistently.

 Props donmhico, Otto42, sproutchris, iandunn, dd32, DaveWP196,
 sebastian.pisula, tommix, sablednah, julian.kimmig, Rahe, clayisland,
 arenddeboer, nicole2292, nagoke, squarecandy, eatingrules, SergeyBiryukov.
 Fixes #21095, #23578, #44589.
 }}}

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


More information about the wp-trac mailing list