Make WordPress Core

Opened 3 years ago

Last modified 2 years ago

#55140 new defect (bug)

Erase Personal Data Email Validation Issue

Reported by: ravipatel's profile ravipatel Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Privacy Keywords: has-patch 2nd-opinion has-privacy-review
Focuses: administration Cc:

Description

Erase Personal Data work fine with invalid username but not working with email.

When i added random text email auto inserted.

Attachments (4)

admin_email.png (127.1 KB) - added by ravipatel 3 years ago.
Email validation issue
55140.patch (926 bytes) - added by ravipatel 3 years ago.
Create a validation code for email
admin_after_validation.png (109.3 KB) - added by ravipatel 3 years ago.
Admin email validation after patch code
55140.2.patch (949 bytes) - added by dilipbheda 3 years ago.

Download all attachments as: .zip

Change History (13)

@ravipatel
3 years ago

Email validation issue

@ravipatel
3 years ago

Create a validation code for email

@ravipatel
3 years ago

Admin email validation after patch code

#1 @ravipatel
3 years ago

  • Keywords has-patch added

Step 1: Add invalid user show validation message
Step 2: When add email with random text auto inserted without validation (bug)
step 3: Need to add validation code on else part.

#2 @mukesh27
3 years ago

  • Component changed from General to Privacy
  • Severity changed from major to normal
  • Version trunk deleted

@dilipbheda
3 years ago

#3 @dilipbheda
3 years ago

@ravipatel I have added below change in updated patch.

1) Remove extract space before elseif
2) Use email field instead of user_email - Ref( https://developer.wordpress.org/reference/functions/get_user_by/ )

#4 @swissspidy
3 years ago

  • Keywords 2nd-opinion added

AFAIK the data removal also needs to support erasing data from guest users who have commented on your site but are not registered users on that site. Or users who have bought something in your ecommerce shop without signing up.

This patch prevents this from happening.

#5 @swissspidy
3 years ago

  • Keywords close added

#6 @SergeyBiryukov
3 years ago

  • Focuses coding-standards removed
  • Keywords needs-privacy-review added

This ticket was mentioned in Slack in #core-privacy by paapst. View the logs.


2 years ago

This ticket was mentioned in Slack in #core-privacy by paaljoachim. View the logs.


2 years ago

#9 @paapst
2 years ago

  • Keywords has-privacy-review added; close needs-privacy-review removed

We discussed this in Core-Privacy today. The patch seems to be on the right path.

Note: See TracTickets for help on using tickets.