[PATCH] Include all email headers in mboxes

Daniel Axtens dja at axtens.net
Fri Apr 6 01:47:19 AEST 2018


>> > That said, using a dict for this is in general not quite right, since
>> > many header lines are valid multiple times, e.g. "Received:", but I'm
>> > not sure they're even all stored.
>> 
>> Right, you and Veronika have convinced me. Clearly as a developer of
>> patchwork my view is a bit skewed - I have to look at them fairly
>> frequently when people report parsing bugs so I forget that other people
>> don't do this.
>> 
>> Veronika - can you check on repeated headers issue that Johannes raises?
>> 
>
> Right, this won't be too friendly in case of duplicate header keys. We do
> store all headers with duplicate keys so I can easily change that and post
> updated patch shortly.
>
> However, since this was brought up, the same situation is present in
> the API -- only one of the duplicate headers is shown. Based on my testing,
> the code on our side does return correct values so this should be a problem
> with the REST framework or something similar. I might take a look at it
> later but can't promise I'll be able to fix it easily.

Thanks! Fixing one problem at a time is fine - feel free to leave the
API for later. I think it'll need a special serializer so we may need
Stephen's expertise.

Regards,
Daniel


More information about the Patchwork mailing list