WordPress.org

Make WordPress Core

Opened 4 months ago

Last modified 4 months ago

#47557 new enhancement

Sanitize Email Suggestion

Reported by: dandersoncm Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 5.2.1
Component: Formatting Keywords: needs-patch needs-unit-tests
Focuses: Cc:
PR Number:

Description

I am using WooCommerce and I've noticed several customer emails come through like...

example@example.com1234
example@example.com1234567812345678

It's mostly due to the email input being the last one before the credit card step, but these emails are passing the validation and sanitization that exists: is_email and sanitize_email.

I am doing something like the following to fix...

<?php

    public function clean_billing_email_address( $value ) {
        return trim( preg_replace( '/\d*$/', '', $value ) );
    }
    add_filter( 'woocommerce_process_checkout_field_billing_email', 'clean_billing_email_address' );

You may consider adding something like this to the sanitize_email function since no TLD ends with numbers anyways, at least at this point in time.

Change History (2)

#1 @mostafa.s1990
4 months ago

  • Keywords needs-patch added

#2 @SergeyBiryukov
4 months ago

  • Keywords needs-unit-tests added
Note: See TracTickets for help on using tickets.