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

Re: How do I make git-format-patch reject 8-bit encodings?

From
TTTimur Tabi <timur@freescale.com>
Date
Feb 22, 2011, 14:49 UTC
Message-ID
<4D63CCDF.50803@freescale.com>
In-Reply-To
<20110222073330.GA8626@sigill.intra.peff.net>
Jeff King wrote:
> Can you provide an example of a message that is base64-encoded? I didn't
> think we base64-encoded anything in send-email (we do rfc2047-encode
> 8-bit header lines, but using quoted-printable). And looking through it,
> I don't see any code to do so.

I can forward to you the original email that is base64-encoded, but I cannot reproduce the problem. Even when I take that patch, apply it with git-am and recreate it with git-format-patch, when I sent that recreate patch with git-send-email, it is not base64 encoded.

> It's possible that I'm missing the code. Or that one of the underlying
> modules is doing it for us. Or it's possible that one of the SMTP
> servers in your route is doing it.

It is possible that our mail server is doing it. We recently migrated to the "Microsoft cloud" email service, and we've experienced all sorts of odd problems with mailing patches.

> If you can send an example of original format-patch output that you fed
> to git-send-email, and the resulting mail that was delivered, then we
> can know more.
I don't know if I can still get the original patch that was encoded.
Show 6 quoted lines
> I have a suspicion it is the last one (some MTA doing it), because git
> tends to generate messages with an 8bit transfer encoding. If we hit a
> server that doesn't advertise support for 8BIT SMTP extensions, I
> believe the sending MTA is required to encode (or bounce). That would
> also account for the inconsistency you noted. It depends on the
> recipient and the exact route of SMTP servers in the delivery.
Ah, this might be a clue.  I will do more research and get back to you.
-- 
Timur Tabi
Linux kernel developer at Freescale
Previous: Jeff King
Message 5 of 5 in “How do I make git-format-patch reject 8-bit encodings?”
  1. Timur TabiFeb 21, 2011
  2. Junio C HamanoFeb 22, 2011
  3. Tabi Timur-B04825Feb 22, 2011
  4. Jeff KingFeb 22, 2011
  5. Timur TabiFeb 22, 2011

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.