[Bug 48795] New: Patches website mishandles HTML attachments
WineHQ Bugzilla
wine-bugs at winehq.org
Mon Mar 23 06:19:39 CDT 2020
https://bugs.winehq.org/show_bug.cgi?id=48795
Bug ID: 48795
Summary: Patches website mishandles HTML attachments
Product: WineHQ.org
Version: unspecified
Hardware: x86
OS: Linux
Status: NEW
Severity: normal
Priority: P2
Component: www-unknown
Assignee: wine-bugs at winehq.org
Reporter: fgouget at codeweavers.com
Distribution: ---
Here is an example:
https://source.winehq.org/patches/data/181590
Created from:
https://www.winehq.org/pipermail/wine-devel/2020-March/162581.html
This typically happens with replies to patches:
* In the plain text version the diff is prefixed with "> " characters so it is
not recognized as a patch.
* But in the HTML version or the email the quoting is done through HTML tags
which typically leave "+++" at the start of the line. As a result the HTML part
can look like a patch.
But it could also happen if someone sends a patch in an HTML-only email (e.g.
due to a mail client misconfiguration).
The TestBot normally ignores HTML attachments (see NewPatch() in Patches.pm).
But when patches go through the patches website the TestBot does not have
access to the raw email. Instead it gets the files prepared by the patches
website as in the first link. Such files erase the distinction between the
plain text and HTML part and so mislead the TestBot into creating a job.
--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.
More information about the wine-bugs
mailing list