search

jackbearheart / email-addresses

An RFC 5322 email address parser
MIT License
249 stars 36 forks source link

Allow comma in display name #54

Closed osm closed 3 years ago

jackbearheart commented 3 years ago

Hey, thanks for the PR.

A goal of this project is to implement email address parsing as per the specs published in RFCs. The comment you edited isn't a line we wrote, but is the original spec from rfc 5322. Unless you know of another document that supersedes this and allows commas, we can't just edit that.

However, I understand that this exception might be useful to people. We could add it to this library if we hide it behind an option flag.

osm commented 3 years ago

Thanks for great feedback. I have now published a new commit that adds a new option called commaInDisplayName, which works in a similar way as the atInDisplayName-option.

osm commented 3 years ago

Hi,

Just wanted to ping you @jackbearheart in case you missed the new commits in the PR.

jackbearheart commented 3 years ago

@osm Sorry for the delay. Thanks for this PR, it looks good!