Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#33328 closed defect (bug) (duplicate)

Retrieve password message fail when sending html messages

Reported by: timersys Owned by:
Priority: normal Milestone:
Component: General Version: 4.3
Severity: normal Keywords:
Cc: Focuses:

Description

When you set up your WordPress to send HTML templates the reset password link fails on almost every email client.

The solution is simple to remove "<" and ">" from the link in

$message .= '<' . network_site_url("wp-login.php?action=rp&key=$key&login=" . rawurlencode($user_login), 'login') . ">\r\n";

Attachments (1)

patch.33328.diff (746 bytes ) - added by timersys 11 years ago.

Download all attachments as: .zip

Change History (10)

#1 @obenland
11 years ago

  • Milestone Awaiting Review4.3
  • Owner set to markjaquith
  • Status newreviewing

@markjaquith do you have time to investigate that?

#2 @obenland
11 years ago

Introduced in [16285] for #14140.

#3 @timersys
11 years ago

It seems that is more related to #23578 . I knew that those brackets were there for a reason, I just couldn't find it

Last edited 11 years ago by SergeyBiryukov (previous) (diff)

#4 @ocean90
11 years ago

Related/duplicate: #23420, #21095

#5 @ocean90
11 years ago

  • Focuses accessibility removed

#6 @obenland
11 years ago

This might not be something we need to fix in 4.3 then.

#7 @timersys
11 years ago

The patch provided in #23578 is simple and effective and it was submitted 2 years ago.... wrapping emails in nice html templates it's pretty common nowadays. While this is not urgent / critical, I don't think implementing that patch this will take more time than reading my messages :)

Thanks!

Last edited 11 years ago by SergeyBiryukov (previous) (diff)

#8 @obenland
11 years ago

  • Milestone 4.3Awaiting Review
  • Owner markjaquith removed

#9 @SergeyBiryukov
11 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status reviewingclosed

Duplicate of #23578.

Note: See TracTickets for help on using tickets.