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

Re: [PATCH] parser: Unmangle From: headers that have been mangled for DMARC purposes

From
ADAndrew Donnellan <ajd@linux.ibm.com>
Date
Oct 11, 2019, 04:36 UTC
Message-ID
<c942d9ce-d8fe-32ca-bedd-1cdb3837823d@linux.ibm.com>
In-Reply-To
<xmqqblunj461.fsf@gitster-ct.c.googlers.com>
On 11/10/19 3:29 pm, Junio C Hamano wrote:
Show 11 quoted lines
> Jeff King <peff@peff.net> writes:
> 
>> This might provide an alternate solution (or vice versa). I kind of like
>> this one better in that it doesn't require the sender to do anything
>> differently (but it may be less robust, as it assumes the receiver
>> reliably de-mangling).
> 
> I share the assessment.  I also feel that relying on Reply-To: would
> make the result a lot less reliable (I do not have much problem with
> the use of X-Original-Sender, though).
>

It would be nice if Mailman could adopt X-Original-Sender too. As it is, it adds the original sender to Reply-To, but in some cases (where the list is set as reply-to-list, or has a custom reply-to setting) it adds to Cc instead. (In the patch that started this thread, I match the name from the munged From field against the name in Reply-To/Cc for the case where there's multiple Reply-Tos/Ccs.)

For the Patchwork use case, I'm quite okay with accepting the risk of using Reply-To, as the alternative is worse, the corner cases are rare, and ultimately a maintainer can still fix the odd stuff-up before applying the patch.

-- 
Andrew Donnellan              OzLabs, ADL Canberra
ajd@linux.ibm.com             IBM Australia Limited
Previous: Junio C HamanoNext: Andrew Donnellan
Message 12 of 15 in “Re: [PATCH] parser: Unmangle From: headers that have been mangled for DMARC purposes”
  1. Jonathan NiederOct 10, 2019
  2. Andrew DonnellanOct 10, 2019
  3. Daniel AxtensOct 10, 2019
  4. Stephen RothwellOct 10, 2019
  5. Jeff KingOct 10, 2019
  6. Andrew DonnellanOct 10, 2019
  7. Jeff KingOct 10, 2019
  8. Daniel AxtensOct 11, 2019
  9. Jeff KingOct 11, 2019
  10. Andrew DonnellanOct 13, 2019
  11. Junio C HamanoOct 11, 2019
  12. Andrew DonnellanOct 11, 2019
  13. Andrew DonnellanOct 11, 2019
  14. Christian SchoenebeckOct 11, 2019
  15. Ian KellingOct 11, 2019

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.