[Bug 48795] New: Patches website mishandles HTML attachments
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(a)winehq.org Reporter: fgouget(a)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.
https://bugs.winehq.org/show_bug.cgi?id=48795 --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- Created attachment 66700 --> https://bugs.winehq.org/attachment.cgi?id=66700 Test mbox containing an email with an HTML attachment To use this test mbox, set up ~/.procmailrc to invoke patches/filter and put the resulting file in the relevant maildir directory. Then: formail -ds procmail ~/.procmailrc <~/html.mbox Note that the resulting file will not be added to the patches index because it is missing the "---" line that patches/updates looks for. To get the TestBot to take the file into account run: ./testbot/bin/WinePatchesWebSubmit.pl -- 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.
https://bugs.winehq.org/show_bug.cgi?id=48795 --- Comment #2 from François Gouget <fgouget(a)codeweavers.com> --- Created attachment 66701 --> https://bugs.winehq.org/attachment.cgi?id=66701 File generated from html.mbox -- 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.
http://bugs.winehq.org/show_bug.cgi?id=48795 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |ABANDONED Status|NEW |RESOLVED --- Comment #3 from Alexandre Julliard <julliard(a)winehq.org> --- The patches website was replaced by Gitlab. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=48795 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Closing abandoned bugs. -- 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.
participants (1)
-
WineHQ Bugzilla