[FIX]Parity: handle bare <meta> tags with no <html><head>

RESOLVED FIXED in mozilla1.1beta

Status

()

Core
HTML: Parser
P2
normal
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: jesup, Assigned: bz)

Tracking

Trunk
mozilla1.1beta
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
Mozilla RC1

This site just displays the <meta> tag as text.  This URL is built into all
DirectTV-DSL gateways as the default adddress for the membership site.  (Browse
to 10.5.1.2 (the modem), click on the "Membership Site" link.)  Apparently,
other browsers (IE, maybe ns4.x, maybe Opera, etc) consider the <META> to be in
the head.

This is probably both a 4xp/ie parity issue and a evangelism bug.  They
shouldn't do it, perhaps we should handle it.

This is what that site returns when I telnet and do "GET /Default.asp HTTP/1.0":

HTTP/1.1 200 OK
Server: Microsoft-IIS/5.0
Date: Sat, 11 May 2002 06:23:27 GMT
Last-Modified: Fri, 03 May 2002 03:42:46 GMT
Connection: Keep-alive
Content-Length: 56

<META HTTP-EQUIV="Refresh" CONTENT="0;URL=/index.jsp">
(Reporter)

Updated

16 years ago
Keywords: 4xp
The response purports to be a HTTP 1.1 response, but it lacks the Content-Type
header. I think the content doesn't go to the HTML parser at all. (I think this
should be evang.)
Created attachment 83204 [details] [diff] [review]
Patch to detect that crap as HTML

Mutter content type mutter mutter.

Comment 3

16 years ago
Comment on attachment 83204 [details] [diff] [review]
Patch to detect that crap as HTML

sr=darin
Attachment #83204 - Flags: superreview+
Taking this too.
Assignee: harishd → bzbarsky
Priority: -- → P2
Summary: Parity: handle bare <meta> tags with no <html><head> → [FIX]Parity: handle bare <meta> tags with no <html><head>
Target Milestone: --- → mozilla1.1beta

Comment 5

16 years ago
Comment on attachment 83204 [details] [diff] [review]
Patch to detect that **** as HTML

i talked w/ people about eventually cleaning this code up but until then...
Attachment #83204 - Flags: review+
Fixed on trunk.  jesup, did you want this on branch?
no time to pursue approval and no one seems to really care about this landing on
branch, so marking fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.