New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generates invalid Reply-To header in some situations #33
Comments
The detailed log on the server says:
Apparently the RFC do not allow to encode everything but only the name... so this is a bug in git-multimail (or the underlying Python modules). |
Previous revisions used to leave the complete field unencoded (see merge request #18). The Python modules to do the encoding are not very convenient to use (IIRC, by default it encoded even the field name, and used QP unconditionally even for pure ascii content), and I already had to fight a bit to get it mostly right. I guess we'll have to special case From, To and Reply-To and email-parse them before encoding. No time to work on this right now, sorry. |
This issue is fixed by 13e66ca. |
FWIW I have setup git-multimail on git.debian.org and many people are now using it. I just got a report of some invalid mail generated by git-multimail. When a user pushed this commit: http://anonscm.debian.org/gitweb/?p=lintian/lintian.git;a=commit;h=3bbaa34ddf253dfdd9aee041e8d571a97e73584e
He got back a bounce saying "550 Message headers fail syntax check" (this is exim giving back this answer) and the only thing that looks weird in the mail is the Reply-to header:
Any idea what's going wrong here?
Note that git-format-patch outputs “From: =?UTF-8?q?Bastien=20ROUCARI=C3=88S?= roucaries.bastien@gmail.com” so it definitely encodes the identity/email more wisely.
The text was updated successfully, but these errors were encountered: