WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 11 months ago

#34483 closed defect (bug) (fixed)

Allow apostrophes in email addresses when accounts are registered

Reported by: tomdxw Owned by: boonebgorges
Milestone: 4.8 Priority: normal
Severity: normal Version: 4.3.1
Component: Login and Registration Keywords: has-patch needs-testing
Focuses: Cc:

Description

#18039 addressed adding users via the admin interface.

This ticket is about users registering their own accounts via /wp-login.php?action=register

Attachments (2)

patch.diff (636 bytes) - added by tomdxw 2 years ago.
Patch against trunk
patch.2.diff (638 bytes) - added by tomdxw 2 years ago.
Updated patch to comply with WP coding standards

Download all attachments as: .zip

Change History (8)

@tomdxw
2 years ago

Patch against trunk

#1 @tomdxw
2 years ago

  • Keywords has-patch added

#2 @swissspidy
2 years ago

  • Component changed from General to Login and Registration
  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to 4.4

Could you swap wp_unslash($_POST['user_email']) with wp_unslash( $_POST['user_email'] ) because of coding standards?

@tomdxw
2 years ago

Updated patch to comply with WP coding standards

#3 @boonebgorges
2 years ago

  • Keywords 4.5-early added
  • Milestone changed from 4.4 to Future Release

Yup, but too late for 4.4.

#4 @tomdxw
11 months ago

  • Keywords 4.8-early added; 4.5-early removed

#5 @boonebgorges
11 months ago

  • Keywords 4.8-early removed
  • Milestone changed from Future Release to 4.8
  • Owner set to boonebgorges
  • Status changed from new to assigned

@tomdxw - Sorry this got lost in the shuffle.

#6 @boonebgorges
11 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 39544:

Allow apostrophes in email address during wp-login.php registration.

See #18039 for a related fix when creating users via the Dashboard.

Props tomdxw.
Fixes #34483.

Note: See TracTickets for help on using tickets.