Re:Show HN: IOU – Split shared expenses
by Freak_NL on Wednesday August 21, 2019 @08:15AM
(#20756577)
Please read up on present day email address validation. If you have a regex that is doing more than ^.+@.+$, then you are doing it wrong.
Don't validate the address beyond looking for an '@' in the string; just send an email with a confirmation link to see if it actually works.
Re:Show HN: IOU – Split shared expenses
by shhsshs on Wednesday August 21, 2019 @09:33AM
(#20756967)
I prefer the true email validation regex: https://regex101.com/r/iE0rF5/1
Re:Show HN: IOU – Split shared expenses
by Freak_NL on Wednesday August 21, 2019 @12:29PM
(#20758815)
I feel bad for the developers who run into that monstrosity and actually implement it.
Re:Show HN: IOU – Split shared expenses
by dalbotex on Wednesday August 21, 2019 @08:23AM
(#20756617)
100% this!
For reference, I didn't try any of the more obscure email address features, such as comments or quotation. My address simply has the form [email protected]
> just send an email with a confirmation link to see if it actually works.
Please don't enable anyone to just send emails to arbitrary email-addresses through you. That's how your email/domain will quickly get marked for SPAM.
At the very least, put in place a rate-limit per email-id (and source-ip too maybe). Otherwise i can simply keep entering email-ids of people to spam.
Why the -1's ? Isn't this a genuine concern to avoid getting on a SPAM blacklist? What am i missing?...
I use:
^[^@]+@[^.]+\..+$
for a bit of sanity checking...
Re:Show HN: IOU – Split shared expenses
by dalbotex on Wednesday August 21, 2019 @10:35AM
(#20757593)
Multiple @ symbols are absolutely valid, as long as they are properly quoted ;)
Re:Show HN: IOU – Split shared expenses
by janberchtold on Wednesday August 21, 2019 @08:21AM
(#20756611)
Thanks we will change this!
1. Can you send an email to [email protected] so I can check you address and fix the bug? 2. We will be building mobile apps as soon as possible with this functionality.