Use the last occurence of @ in email normalization

portier / portier.github.io

Website for Portier, an email-based, passwordless authentication that you can host yourself.

https://portier.github.io

364 stars 18 forks source link

Use the last occurence of @ in email normalization #38

Closed jvasseur closed 5 years ago

jvasseur commented 6 years ago

The user part of an email can contains @ when using quoted form where the domain part can't, so spiting on the last occurrence would allow to support more email addresses.

onli commented 5 years ago

I think you are right (sorry, I missed this PR before). There is the issue of the spec diverging from the implementation, but better to start here than to ignore it.

jvasseur commented 5 years ago

I wanted to see if the change was accepted before working on updating the implementation. I will do it soon (the change isn't really hard to do).