The default bug view has changed. See this FAQ.

KHTML-based browsers such as Konqueror do not support the Server-Push technology

RESOLVED FIXED in Bugzilla 4.2

Status

()

Bugzilla
Query/Bug List
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: Frédéric Buclin, Assigned: Frédéric Buclin)

Tracking

Bugzilla 4.2
Bug Flags:
approval +
approval4.4 +
approval4.2 +

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Konqueror and other KHTML-based browsers such as Safari do not support the Server-Push technology used in buglists. We already catch Safari correctly thanks to "WebKit" being in its UA string but we totally miss Konqueror. Side-effects are the inability to use the QuickSearch feature more than once in a row (on the 2nd attempt, the QS stops working) and the UI is broken in guided form when filing a new bug, see e.g. https://bugs.gentoo.org/show_bug.cgi?id=380617.
(Assignee)

Comment 1

4 years ago
Created attachment 801661 [details] [diff] [review]
patch, v1
Attachment #801661 - Flags: review?(dkl)
Comment on attachment 801661 [details] [diff] [review]
patch, v1

Review of attachment 801661 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #801661 - Flags: review?(dkl) → review+

Updated

4 years ago
Flags: approval?
Flags: approval4.4?

Updated

4 years ago
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
(Assignee)

Comment 3

4 years ago
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified buglist.cgi
Committed revision 8733.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified buglist.cgi
Committed revision 8604.
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

4 years ago
approval4.2 = justdave on IRC.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.2/
modified buglist.cgi
Committed revision 8227.
Flags: approval4.2+
Target Milestone: Bugzilla 4.4 → Bugzilla 4.2

Comment 5

3 years ago
Comment on attachment 801661 [details] [diff] [review]
patch, v1

>Index: buglist.cgi
>===================================================================
>RCS file: /cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v
>retrieving revision 1.457
>diff -p -u -r1.457 buglist.cgi
>--- buglist.cgi	13 Aug 2013 18:57:12 -0000	1.457
>+++ buglist.cgi	9 Sep 2013 17:08:57 -0000
>@@ -105,7 +105,7 @@ my $serverpush =
>     && exists $ENV{'HTTP_USER_AGENT'} 
>       && $ENV{'HTTP_USER_AGENT'} =~ /(Mozilla.[3-9]|Opera)/
>         && $ENV{'HTTP_USER_AGENT'} !~ /compatible/i
>-          && $ENV{'HTTP_USER_AGENT'} !~ /(?:WebKit|Trident)/
>+          && $ENV{'HTTP_USER_AGENT'} !~ /(?:WebKit|Trident|KHTML)/
>             && !defined($cgi->param('serverpush'))
>               || $cgi->param('serverpush');
>
You need to log in before you can comment on or make changes to this bug.