#59156 closed enhancement (wontfix)
sanitize_email method should lowercase the given email
Reported by: | achchu93 | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | minor | Version: | 6.3 |
Component: | Formatting | Keywords: | reporter-feedback |
Focuses: | Cc: |
Description
Currently sanitize_email
method doesn't lowercase the given email. An email should in lowercase letters. As this is a formatting method I believe lowercasing it will help out.
Change History (5)
#2
@
13 months ago
Okay, lets say I am sending 'Ahamed@…' as my email. This method sanitize and returns it with the capital letter it already had. If I want to save the email as a meta, I have to lowercase it again. Lets say if I have saved the email in Capital letters I have to use this method and again I have to run lowercase function. As this is a formatting helper function it would be great if we can get the output in lowercase. So we don't have to use strtolower function everywhere in the plugin. One single formatting function will do whatever it requires for the email.
#3
@
13 months ago
As far as I know, Capitalization is allowed in email addresses. It is usually ignored (like WP does), but the official standard does not require email addresses to be lowercase.
Since sanitize_email
is intended to make sure an email address conforms to the standard, I do not believe it should convert the input to lowercase.
#4
@
11 months ago
- Resolution set to wontfix
- Status changed from new to closed
@roytanck Correct, see https://datatracker.ietf.org/doc/html/rfc3696#section-4.1
therefore these are valid, examples taken from the RFC
Joe@example.com user+mailbox@example.com customer/department=shipping@example.com $A12345@example.com !def!xyz%abc@example.com _somename@example.com
Hi @achchu93, welcome to WordPress Trac, and thanks for your ticket.
Can you please explain the necessity of this feature? Because currently, all the emails are always treated in lowercase. So, for example, you have a user with the email "rajin@…". After that, when you try to create another user with "RAJIN@…", it will show the error that you are trying to use an already used email.
Also, if you try to log in with "RAJIN@…", you will be able to log in to the account. So, I am not quite sure what's the necessity of this proposed function. Feel free to add your thoughts here.