systopia / CiviProxy

A security proxy for CiviCRM
GNU Affero General Public License v3.0
7 stars 18 forks source link

Email validation regex is not matching '+' in the user part #35

Open d1ss0nanz opened 4 years ago

d1ss0nanz commented 4 years ago

foo.bar+label@gmail.com isn't accepted as valid email.

bjendres commented 4 years ago

fair point. @d1ss0nanz would you care to create a PR?

d1ss0nanz commented 4 years ago

Sure. Any thoughts on using filter_var instead of a custom regex?

bjendres commented 4 years ago

Seeing that this project is all about a minimal attack vector: do you think filter_var is safer than preg_match?