git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] git-send-email: Add --no-validate-email option

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 30, 2022, 21:03 UTC
Message-ID
<xmqqzghtdftx.fsf@gitster.g>
In-Reply-To
<220630.868rpee6d3.gmgdl@evledraar.gmail.com>
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 13 quoted lines
> I'm suggesting that we replace our own validation with that of the SMTP
> server's, yes they're don't 1=1 correspond, but I think the part of the
> Venn diagram of where that matters is too small to worry about.
>
> It has the advantage of side-stepping issues with not having
> Email::Valid, as well as those cases where we're being overzelous about
> RFC validation, but our local SMTP is willing to try to deliver the
> mail.
>
> It's not like authors of MTAs haven't heard of that character limit, but
> they're also aware that that certain parts of the spec are loosely
> enforced, and that trying delivery is often better than rejecting a mail
> out of RFC pedantry.
I am not sure if that is a healthy direction to go.

If a local outbound relay is written with the knowledge that it will never be talking to the SMTP at the final mailbox directly, I would expect that it may not implement any validation at all, relying on the "next hop" smarthost to reject anything invalid it throws at it.

So...
Previous: Ævar Arnfjörð Bjarmason
Message 8 of 8 in “git-send-email: Add --no-validate-email option”
  1. git-send-email: Add --no-validate-email optionStewart Smith, Jun 20, 2022
  2. Ævar Arnfjörð BjarmasonJun 20, 2022
  3. brian m. carlsonJun 21, 2022
  4. Junio C HamanoJun 21, 2022
  5. Ævar Arnfjörð BjarmasonJun 21, 2022
  6. brian m. carlsonJun 22, 2022
  7. Ævar Arnfjörð BjarmasonJun 30, 2022
  8. Junio C HamanoJun 30, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.