Stephen Finucane <step...@that.guru> writes:

> With a recent change, we started using the original subject header
> instead of the one we had already cleaned up at the parsing stage.

Does anyone care about the cleaned up header in the mbox? Git strips it
all off anyway... Is there any docs on why we did things that way
originally?

Regards,
Daniel

> Revert this aspect of that change.
>
> Signed-off-by: Stephen Finucane <step...@that.guru>
> Fixes: 01b9cbb9 ("Include all email headers in mboxes")
> ---
>  patchwork/views/utils.py | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/patchwork/views/utils.py b/patchwork/views/utils.py
> index f5ff43c1..7f89004b 100644
> --- a/patchwork/views/utils.py
> +++ b/patchwork/views/utils.py
> @@ -101,6 +101,9 @@ def _submission_to_mbox(submission):
>      for key, val in orig_headers.items():
>          mail[key] = val
>  
> +    # specifically overwrite the subject with our own nicely formatted name
> +    mail['Subject'] = submission.name
> +
>      if 'Date' not in mail:
>          mail['Date'] = email.utils.formatdate(utc_timestamp)
>  
> -- 
> 2.14.3
>
> _______________________________________________
> Patchwork mailing list
> Patchwork@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/patchwork
_______________________________________________
Patchwork mailing list
Patchwork@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/patchwork

Reply via email to