Internet Explorer 11 receives multipart/x-mixed-replace content from buglist.cgi

RESOLVED FIXED in Bugzilla 4.0

Status

()

Bugzilla
Query/Bug List
RESOLVED FIXED
4 years ago
a year ago

People

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

Tracking

4.2.6
Bugzilla 4.0
Bug Flags:
approval +
approval4.4 +
approval4.2 +
approval4.0 +

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

4 years ago
Created attachment 786987 [details]
Request-response pair exported from F12 tool in IE11 showing headers used

User Agent: HTML5

Steps to reproduce:

I clicked the Search tab on this Bugzilla, selected "all" and entered "internet explorer" to the search field, then pressed enter to search this Bugzilla for all bugs related to Internet Explorer 11.


Actual results:

Bugzilla sent the results back to IE11, which offered to save it to a file named buglist. Examining the HTTP traffic shows that Bugzilla was using multipart/x-mixed-replace as a Content-Type, which isn't part of HTML5 and doesn't appear to be standardized anywhere at all.


Expected results:

Bugzilla should have sent the results back using regular HTML5. If asynchronously loaded results are needed for better user experience Bugzilla should have used Server-Sent Events with AJAX fallback for browsers where S-SE are not available.

Using F12 Developer Tools to use a custom user-agent like "HTML5" instead of the default "Mozilla/5.0 (Windows NT 6.3; WOW64; Trident/7.0; rv:11.0) like Gecko" and everything else being the same leads to the search working correctly.
(Reporter)

Comment 1

4 years ago
Created attachment 786988 [details]
Search results received by IE11
(Assignee)

Comment 2

4 years ago
Bugzilla uses HTML 4.01 Transitional, not HTML5, and so doesn't use Server-Sent Events which is part of HTML5. And per http://en.wikipedia.org/wiki/Server-sent_events, Internet Explorer doesn't support this technology anyway.

IE11 gets multipart/x-mixed-replace because its UserAgent string doesn't follow older versions of IE, i.e. both "MSIE" and "compatible" are gone. This breaks our detection of the browser.
(Assignee)

Comment 3

4 years ago
Also note that if IE11 UserAgent string contains "like Gecko", then IE11 should work like Gecko, i.e. support the server-push technology. As suggested on http://msdn.microsoft.com/en-us/library/ie/bg182625%28v=vs.85%29.aspx, we should look for "Trident" to detect IE.
(Assignee)

Comment 4

4 years ago
Created attachment 787053 [details] [diff] [review]
patch, v1

Trident is present in the UserAgent string since IE8.
Assignee: query-and-buglist → LpSolit
Attachment #786987 - Attachment is obsolete: true
Attachment #786988 - Attachment is obsolete: true
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #787053 - Flags: review?(dkl)
Comment on attachment 787053 [details] [diff] [review]
patch, v1

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

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

Updated

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

Updated

4 years ago
Flags: approval4.2+
Flags: approval4.0+

Updated

4 years ago
Target Milestone: --- → Bugzilla 4.0
We were just discussing this on IRC, the reasoning for putting this on the 4.2 and 4.0 branches also is because these branches will still be supported when IE11 releases, and major browser compatibility issues are a big deal.
(Assignee)

Comment 7

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

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/
modified buglist.cgi
Committed revision 8593.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.2/
modified buglist.cgi
Committed revision 8223.

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

Comment 8

4 years ago
Created attachment 8342904 [details]
20131101_153817.jpg

Thank
Flags: needinfo?(LpSolit)

Updated

4 years ago
Attachment #8342904 - Attachment is patch: false
Attachment #8342904 - Attachment mime type: text/plain → image/jpg

Comment 9

4 years ago
Ianiey: Do NOT use bugzilla.mozilla.org for testing, as this is a production Bugzilla!
Use https://landfill.bugzilla.org/ for testing Bugzilla instead!
Flags: needinfo?(LpSolit)
(Assignee)

Comment 10

4 years ago
Comment on attachment 8342904 [details]
20131101_153817.jpg

Some bmo admin please delete this attachment.
Attachment #8342904 - Attachment is obsolete: true
The content of attachment 8342904 [details] has been deleted for the following reason:

mistaken upload

Comment 12

3 years ago
I am also seeing this issue still on v4.4
Flags: needinfo?(LpSolit)
(In reply to admin from comment #12)
> I am also seeing this issue still on v4.4

please file a new bug if you think this issue has not been fixed.  make sure you include both the bugzilla and IE versions.
Flags: needinfo?(LpSolit)
You need to log in before you can comment on or make changes to this bug.