Author Joaquin.Cuenca.Abela
Recipients Joaquin.Cuenca.Abela, athomas, barry, esam, r.david.murray
Date 2010-03-07.18:04:02
SpamBayes Score 1.34371e-06
Marked as misclassified No
Message-id <1267985043.76.0.273788296801.issue7143@psf.upfronthosting.co.za>
In-reply-to
Content
Unfortunately the only way that I can see to reliably work around this is to bypass entirely get_payload, in this case fixing this bug will not affect people that do that negatively.

Some people may have more control over their attachments and they are unconditionally adding a \n to certain type of attachments. In this case they will get an extra \n after this patch.

If they are doing this, they certainly realized there was a bug, because this is only necessary for base64 encoded attachments and not for any other encoding. In this case I guess it's reasonable to expect that this bug will be fixed one day.

But you're right that the change is risky, an that we (at least I) are not sure why the code is the way it is.

Cheers,
History
Date User Action Args
2010-03-07 18:04:03Joaquin.Cuenca.Abelasetrecipients: + Joaquin.Cuenca.Abela, barry, r.david.murray, athomas, esam
2010-03-07 18:04:03Joaquin.Cuenca.Abelasetmessageid: <1267985043.76.0.273788296801.issue7143@psf.upfronthosting.co.za>
2010-03-07 18:04:02Joaquin.Cuenca.Abelalinkissue7143 messages
2010-03-07 18:04:02Joaquin.Cuenca.Abelacreate