Make WordPress Core

Opened 20 months ago

Last modified 12 months 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 20 months ago.
Email validation issue
55140.patch (926 bytes) - added by ravipatel 20 months ago.
Create a validation code for email
admin_after_validation.png (109.3 KB) - added by ravipatel 20 months ago.
Admin email validation after patch code
55140.2.patch (949 bytes) - added by dilipbheda 20 months ago.

Download all attachments as: .zip

Change History (13)

@ravipatel
20 months ago

Email validation issue

@ravipatel
20 months ago

Create a validation code for email

@ravipatel
20 months ago

Admin email validation after patch code

#1 @ravipatel
20 months 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
20 months ago

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

#3 @dilipbheda
20 months 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
20 months 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
20 months ago

  • Keywords close added

#6 @SergeyBiryukov
20 months ago

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

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


12 months ago

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


12 months ago

#9 @paapst
12 months 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.