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