[wp-trac] [WordPress Trac] #23578: URLs wrapped in <> parsed as HTML when wp_mail_content_type set to text/html

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


#23578: URLs wrapped in <> parsed as HTML when wp_mail_content_type set to
text/html
--------------------------+-----------------------------
 Reporter:  iandunn       |       Owner:  SergeyBiryukov
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  Future Release
Component:  Mail          |     Version:  3.1
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |     Focuses:
--------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * owner:  (none) => SergeyBiryukov
 * status:  new => 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/23578#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list