241890 2022-06-22 17:28:09 -0700 bugs.webkit.org returns 500 error when github commit URL is entered into the Comment field. 2022-08-28 14:10:27 -0700 1 1 1 Unclassified WebKit WebKit Website Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=242128 InRadar P2 Normal --- 1 lingho lingho webkit-bug-importer oldest_to_newest 1877503 0 lingho 2022-06-22 17:28:09 -0700 https://github.com/WebKit/WebKit/commit/431f4db465c43b419910c6889d60edb9454a7de5#r76747819 1877600 1 lingho 2022-06-23 01:55:47 -0700 test 1877680 2 lingho 2022-06-23 10:08:37 -0700 https://github.com/WebKit/WebKit/commit/431f4db465c43b419910c6889d60edb9454a7de5#r76747819 1878055 3 lingho 2022-06-23 17:58:19 -0700 As soon as a github.com Commit URL such as https://github.com/WebKit/WebKit/commit/431f4db465c43b419910c6889d60edb9454a7de5#r76747819 is entered into the Comment field, and saved, Bugzilla produces a 500 error. Any subsequence actions on the bug causes the same error. 1878056 4 lingho 2022-06-23 17:59:48 -0700 This error is shown in /var/log/httpd/error_log. [Thu Jun 23 01:55:47.252524 2022] [:error] [pid 11149] "\\x{fdd2}" does not map to utf8 at /usr/lib64/perl5/vendor_perl/Encode.pm line 160.\n 1878057 5 webkit-bug-importer 2022-06-23 18:00:26 -0700 <rdar://problem/95820523> 1878059 6 lingho 2022-06-23 18:01:00 -0700 *** Bug 241891 has been marked as a duplicate of this bug. *** 1878061 7 lingho 2022-06-23 18:01:13 -0700 *** Bug 241892 has been marked as a duplicate of this bug. *** 1878063 8 lingho 2022-06-23 18:01:26 -0700 *** Bug 241893 has been marked as a duplicate of this bug. *** 1878066 9 lingho 2022-06-23 18:01:41 -0700 *** Bug 241896 has been marked as a duplicate of this bug. *** 1878071 10 lingho 2022-06-23 18:14:19 -0700 I was unabled to reproduce this on the UAT server. However, adding the URL in the Comment field, then clicking on Preview shows that the URL is being turned into a link, while the revision is also being converted into a https://commits.webkit.or URL one way or another. The revision part of the link then became https://github.com/WebKit/WebKit/commit/431f4db465c43b419910c6889d60edb9454a7de5#﷒0﷓ This seems to be happenning here in Bugzilla/Template.pm: my ($match, $replace) = @$re{qw(match replace)}; if (ref($replace) eq 'CODE') { $text =~ s/$match/($things[$count++] = $replace->({matches => [ $1, $2, $3, $4, $5, $6, $7, $8, $9, $10]})) && ("\x{FDD2}" . ($count-1) . "\x{FDD3}")/egx; } Fixing the Commit extention to not replace a revision number when it follows a hash prevented this issue, and got rid of the 500 error. 1878079 11 lingho 2022-06-23 18:26:00 -0700 Pull request: https://github.com/WebKit/WebKit/pull/1761 1894339 12 ews-feeder 2022-08-28 14:10:26 -0700 Committed 253885@main (b823352209d3): <https://commits.webkit.org/253885@main> Reviewed commits have been landed. Closing PR #1761 and removing active labels.