Bug 209484 - Add user agent quirk for Red Hat Bugzilla
Summary: Add user agent quirk for Red Hat Bugzilla
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-24 10:38 PDT by Michael Catanzaro
Modified: 2020-03-30 10:12 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.88 KB, patch)
2020-03-24 10:42 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2020-03-24 10:38:34 PDT
Red Hat Bugzilla displays a warning page when performing searches unless we use a Firefox or Google user agent quirk. I've contacted the developers and they do not want to stop abusing the user agent header for this purpose, so let's do the usual.
Comment 1 Michael Catanzaro 2020-03-24 10:42:37 PDT
Created attachment 394381 [details]
Patch
Comment 2 Adrian Perez 2020-03-24 17:08:14 PDT
Comment on attachment 394381 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=394381&action=review

> Source/WebCore/platform/UserAgentQuirks.cpp:98
> +    if (domain == "bugzilla.redhat.com")

I find it a bit odd that a particular instance of Bugzilla
needs a quirk, and others do not need it ¯\_(ツ)_/¯
Comment 3 Alexey Proskuryakov 2020-03-24 22:30:17 PDT
This file looks like it's cross-platform, and so do function names. But this function is only used by WebCore/platform/glib/UserAgentGLib.cpp.
Comment 4 Michael Catanzaro 2020-03-25 06:56:56 PDT
We do occasionally have mostly cross-platform files that don't make sense on one particular platform. The quirks are separated from UserAgentGLib.cpp because they are intended to be entirely cross-platform. It's pretty likely that any websites causing problems for GLib ports will also be causing the same problems for Windows and PlayStation as well, for instance. I would recommend these quirks to any non-Apple port. If it would help avoid confusion, we could guard the whole thing with #if !PLATFORM(COCOA).

(In reply to Adrian Perez from comment #2)
> I find it a bit odd that a particular instance of Bugzilla
> needs a quirk, and others do not need it ¯\_(ツ)_/¯

I see you haven't spent much time on Red Hat Bugzilla. :P It's really its own thing.
Comment 5 EWS 2020-03-30 10:12:04 PDT
Committed r259204: <https://trac.webkit.org/changeset/259204>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 394381 [details].