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

Invalid email address in test_header.js #15

Open
konfusator opened this issue Nov 6, 2015 · 3 comments
Open

Invalid email address in test_header.js #15

konfusator opened this issue Nov 6, 2015 · 3 comments

Comments

@konfusator
Copy link

The address [email protected] <[email protected]> which is introduced to /test/test_header.js in 4d830f6 is syntactically invalid.

RFC 5322 defines @ to be an element of specials, so the display-name has to be a quoted-string, i.e. the right address would be "[email protected]" <[email protected]>

@TheRealCuran
Copy link

Ping?

This is really annoying and breaks parts of the e-mail addresses filled in from my Thunderbird address book.

@lygstate
Copy link

Gocha, maybe I will create a pull request.

@jcranmer
Copy link
Contributor

Actually, the test in question is about parsing headers, so parsing an invalid RFC 5322 production to a sensible representation shouldn't be a problem.

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

4 participants