diff mbox series

[BUG,b4] encoding issues with single-patch series

Message ID CTDYR84I395O.2UI69IKNO7B8A@fedora (mailing list archive)
State Superseded
Headers show
Series [BUG,b4] encoding issues with single-patch series | expand

Commit Message

Erik Schilling June 16, 2023, 9:22 a.m. UTC
Earlier, I sent a patch where I tried to CC Alex Bennée. The generated
mail included the header:

    Content-Transfer-Encoding: 7bit

the CC header got correctly escaped, but the mail body contained the CC
with UTF-8 encoding:

    Cc: Alex Bennée <alex.bennee@linaro.org>


The mail got dispatched without any complaints, but then was rejected by
linux-gpio with:

    <linux-gpio@vger.kernel.org>: host 23.128.96.18[23.128.96.18] said: 550 5.7.1
        Content-Policy reject msg: Wrong MIME labeling on 8-bit character texts.
        BF:<H 5.99033e-10>; S229768AbjFPGtM (in reply to end of DATA command)

One can reproduce it with:

    git clone -b b4/msrv https://github.com/Ablu/libgpiod.git
    cd libgpiod
    b4 send -o /tmp/test/ --resend v1
    # The generated mail includes UTF-8 in the body,
    # but has a 7bit header

I did a quick hack to get it resent, but not sure if that is a proper
solution:



The weird thing is that some past mails where I also CC'd him,
apparently were sent with the right encoding settings automatically [1]:

    Content-Type: text/plain; charset="utf-8"
    Content-Transfer-Encoding: 8bit

[1] https://lore.kernel.org/linux-gpio/20230612-crates_io_publish-v1-0-70988ee9a655@linaro.org/raw

It looks like it works for a series but not for a single patch?

- Erik

Comments

Konstantin Ryabitsev June 16, 2023, 4:53 p.m. UTC | #1
On Fri, Jun 16, 2023 at 11:22:51AM +0200, Erik Schilling wrote:
> Earlier, I sent a patch where I tried to CC Alex Bennée. The generated
> mail included the header:
> 
>     Content-Transfer-Encoding: 7bit
> 
> the CC header got correctly escaped, but the mail body contained the CC
> with UTF-8 encoding:
> 
>     Cc: Alex Bennée <alex.bennee@linaro.org>

Interesting. I'll check it out.

bugbot assign to me

-K
Kernel.org Bugbot June 16, 2023, 4:55 p.m. UTC | #2
Hello:

This conversation is now tracked by Kernel.org Bugzilla:
https://bugzilla.kernel.org/show_bug.cgi?id=217563

There is no need to do anything else, just keep talking.
Konstantin Ryabitsev June 22, 2023, 5:22 p.m. UTC | #3
On Fri, Jun 16, 2023 at 11:22:51AM +0200, Erik Schilling wrote:
> Earlier, I sent a patch where I tried to CC Alex Bennée. The generated
> mail included the header:
> 
>     Content-Transfer-Encoding: 7bit
> 
> the CC header got correctly escaped, but the mail body contained the CC
> with UTF-8 encoding:
> 
>     Cc: Alex Bennée <alex.bennee@linaro.org>

I was able to replicate this, thanks.

bugbot assign to me

-K
Kernel.org Bugbot June 22, 2023, 5:47 p.m. UTC | #4
Konstantin Ryabitsev writes in commit cee377e23203fd12590316c96ceadae8357e7a77:

Fix wrong CTR header after 8bit cover content is mixed in

When we mix in cover content into the patch that previously only had
7bit data, re-check the new payload and set C-T-R to 8bit if the new
payload is no longer ascii.

Reported-by: Erik Schilling <erik.schilling@linaro.org>
Closes: https://bugzilla.kernel.org/show_bug.cgi?id=217563
Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>

(via https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=cee377e23203)
diff mbox series

Patch

--- a/b4/ez.py
+++ b/b4/ez.py
@@ -1581,6 +1581,9 @@  def cmd_send(cmdargs: argparse.Namespace) -> None:
             else:
                 msg.add_header('Cc', b4.format_addrs(pcc))
 
+        msg.set_charset('utf-8')
+        msg.replace_header('Content-Transfer-Encoding', '8bit')
+
         send_msgs.append(msg)
 
     if endpoint: