Comment on I'm locked out of my 6 year old Chipotle account because they now say my email address is invalid when I login. Here is me asking for their help:

• xantoxis@lemmy.world ⁨1⁩ ⁨year⁩ ago

  Sorry, this is not a correct regex for an email address.

  Sending using ‘mail’ on a local unix system? You only need the local part.

  STOP VALIDATING NAMES AND EMAIL ADDRESSES. Send a verification email. Full stop. Don’t do anything else. You really want to do this anyway, because it’s a defense against spam.

  source

  • stardreamer@lemmy.blahaj.zone ⁨1⁩ ⁨year⁩ ago

    *Gasp* the registration is coming from inside the data center!

    source
  • Turun@feddit.de ⁨1⁩ ⁨year⁩ ago

    I think it’s fair to prevent users from causing mail sent to your internal systems. It probably won’t cause any issues getting mail to the machine inbox for (no domain name), but it reasonably makes security uneasy.

    source

    • xantoxis@lemmy.world ⁨1⁩ ⁨year⁩ ago

      The statement I was responding to was “This is the correct email regex”. There is no correct email regex. Don’t parse emails with a regex. You probably don’t need to parse emails at all.

      source
  • elrik@lemmy.world ⁨1⁩ ⁨year⁩ ago

    Yes, but no. Pretty much every application that accepts an email address on a form is going to turn around and make an API call to send that email. Guess what that API is going to do when you send it a string for a recipient address without an @ sign? It’s going to refuse it with an error.

    Therefore the correct amount of validation is that which satisfies whatever format the underlying API requires.

    For example, AWS SES requires addresses in the form UserName@[SubDomain.]Domain.TopLevelDomain along with other caveats. If the application is using SES to send emails, I’m not going to allow an input that doesn’t meet those requirements.

    source

    • xantoxis@lemmy.world ⁨1⁩ ⁨year⁩ ago

      Therefore the correct amount of validation is that which satisfies whatever format the underlying API requires.

      You mean the validation which the underlying API will perform on its own? You don’t need to do it.

      source

      • elrik@lemmy.world ⁨1⁩ ⁨year⁩ ago

        I disagree. You should have validation at each layer, as it’s easier to handle bad inputs and errors the earlier they are caught.

        It’s especially important in this case with email because often one or more of the following comes into play when you’re dealing with an email input:

        • You’re doing more than sending an email (for ex, creating a record for a new user).
        • The UI isn’t waiting for you to send that email (for ex, it’s handled through a queue or some other background process).
        • The API call to send an email has a cost (both time and money).
        • You have multiple email recipients (better hope that external API error tells you which one failed).

        I’m not suggesting that validation of an email should attempt to be exhaustive, but a well thought-out implementation validates all user inputs. Even the underlying API in this example is validating the email you give it before trying to send an email through its own underlying API.

        Passing obvious garbage inputs down is just bad practice.

        source
• tomaThomas@lemmy.world ⁨1⁩ ⁨year⁩ ago

  Here’s my address: @@@@@

  source

  • laurelraven@lemmy.blahaj.zone ⁨1⁩ ⁨year⁩ ago

    And this right here is a great example of why simple basic RegEx is rarely adequate

    At the very least, should be something like

    ^[^@\s]+@([^@\s.]+\.)+[^@\s.]+$

    I’m like 99% sure I missed at least a few cases there, and will say “please don’t use this for anything production”

    source

    • jpeps@lemmy.world ⁨1⁩ ⁨year⁩ ago

      Here’s two: you can have multiple @s forming relays in an email address, and you can also break all the rules around dots and spaces if you put quotes around the local part, eg “.sarah…”@emails.com

      source

      • laurelraven@lemmy.blahaj.zone ⁨1⁩ ⁨year⁩ ago

        And this is exactly why I wouldn’t do my own, I had no idea either of those were legal/possible

        source

        • -> View More Comments