Page MenuHome GnuPG
Create Task
Maniphest T4987

GpgOL breaks URLs by inserting a line break after column 71 in text-only messages
Closed, ResolvedPublic
Actions

Description

Tested with mails from bsi.bund.de, subject e. g. from today
[Buerger-Cert-Warnmeldung] TW-T20-0127 - Mozilla Firefox für Android: Schwachstelle ermöglicht Offenlegung von Informationen

or with emails from a colleague sent with Tobit David

or with all mails from securitynotifications@e-mail.microsoft.com, subject "Microsoft Security Update Releases" e. g. 2020-06-12.
This bug exists already for a very long time (I thought it was a fault on the sender's side since yesterday when I tested :-).

We sent test emails to Office 365 (Enterprise), Outlook.com, gmail and gmx.de in CC - the same email without issues, URLs clickable, if viewed in a web browser and in Outlook 365 as soon as we disable the GpgOL Add-In. Re-Activating breaks the URLs again.

Details

Version
3.1.11

Revisions and Commits

rO GpgOL
rO4db95f1b3934 Put BodyFormat before setting the body

Related Objects
Search...

Event Timeline

OliverL created this task.Jul 8 2020, 5:42 PM
aheinecke claimed this task.Jul 13 2020, 2:03 PM
aheinecke triaged this task as High priority.
aheinecke added a subscriber: aheinecke.

It's not only for URL's. I've tested with any long line when sending "text/plain" GpgOL properly sends this but displays it wrongly.

Strange that this has never been reported before.

OliverL added a comment.Jul 13 2020, 2:39 PM

Yes :-). I see it also in line rows (many '_' characters). As I wrote, I (and probably all others) always exptected this to be a formatting error on the sender's side :-).

aheinecke added a comment.Jul 14 2020, 1:03 PM

After digging through our complete parser code it turns out that we did everything correctly but Outlook adds the line break when we change the bodyformat from HTML to plain text. So this issue only existed since the fix for: T4639

aheinecke changed the task status from Open to Testing.Jul 14 2020, 1:03 PM
aheinecke added a subtask: T4789: Gpg4win-3.1.12.
aheinecke added a commit: rO4db95f1b3934: Put BodyFormat before setting the body.Jul 14 2020, 1:53 PM
aheinecke closed this task as Resolved.Aug 7 2020, 11:15 AM
OliverL added a comment.Aug 8 2020, 2:44 PM

Thank you, Andre. Just updated here, problem fixed.

• werner closed subtask T4789: Gpg4win-3.1.12 as Resolved.Nov 11 2020, 5:02 PM