RESOLVED INVALID 241115
Commit message not showing up in patch review for some patches 
https://bugs.webkit.org/show_bug.cgi?id=241115
Summary Commit message not showing up in patch review for some patches
Alexey Proskuryakov
Reported Tuesday, May 31, 2022 2:52:06 AM UTC
Commit message is not visible in https://bugs.webkit.org/attachment.cgi?id=459876&action=review, although it's present and looks normal in the raw patch, https://bug-241114-attachments.webkit.org/attachment.cgi?id=459876
Attachments
WIP patch (1.36 KB, patch)
2022-05-30 21:26 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 Tuesday, May 31, 2022 5:26:21 AM UTC
Created attachment 459877 [details] WIP patch
Fujii Hironori
Comment 2 Tuesday, May 31, 2022 5:28:50 AM UTC
I'm not able to find out a good way to decode MIME header with Ruby standard library. Subject: =?UTF-8?q?webkit-patch:=20`git=20am`=20strips=20prefixes=20(e.g.?= =?UTF-8?q?=20[CMake][WPE])=20in=20a=20subject=0Ahttps://bugs.webkit.org/s?= =?UTF-8?q?how=5Fbug.cgi=3Fid=3D241114?=
Fujii Hironori
Comment 3 Tuesday, May 31, 2022 5:35:42 AM UTC
`git mailinfo` command can decode the subject. But I don't know git command is available on the Bugzilla server.
Alexey Proskuryakov
Comment 4 Tuesday, May 31, 2022 7:11:34 AM UTC
Running external commands with untrusted content on the server would be icky. I don't know how all this works though, maybe we already do?
Radar WebKit Bug Importer
Comment 5 Tuesday, June 7, 2022 2:53:12 AM UTC
<rdar://problem/94480379>
Fujii Hironori
Comment 6 Tuesday, June 7, 2022 3:14:42 AM UTC
Bug 241114 was fixed by using --keep-non-patch option rather than --keep option. We don't need to fix this problem. Closed.
Note You need to log in before you can comment on or make changes to this bug.