Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Email, maybe a new one? #22

Open
frederikhors opened this issue Dec 7, 2020 · 3 comments
Open

Email, maybe a new one? #22

frederikhors opened this issue Dec 7, 2020 · 3 comments

Comments

@frederikhors
Copy link
Contributor

What do you think about conform's email?

https://github.com/leebenson/conform/blob/master/conform.go#L31-L51

@deankarn
Copy link
Contributor

I don't know if it's necessary TBH, what does manipulating only the domain portion give you if not extracting it?

@frederikhors
Copy link
Contributor Author

I do not understand. Are you saying that checking that part is superfluous?

I'm trying to create something convenient for developers that corrects incorrect or inaccurate user input, e.g.:

CUSTOMER-EMAIL@EXAMPLE.COM" -> "CUSTOMER-EMAIL@example.com"

Because I know according to rfc5321, "The local-part of a mailbox MUST BE treated as case sensitive".

How would you do it?

@deankarn
Copy link
Contributor

I'm just not sure how this would be used? is all caps after the @ actually incorrect?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants