166366 - hbl-miniaturen.de - Redirection URL dowsn't work !

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: German, defect)

Description

[Mirek Hancl]

When opening one of the following URLs, an error occures:
HBL - Home Page <http://www.hbl-miniaturen.de/>
ISE HomePage <http://www.ise-software.com/>
tecnotron software gmbh - Home Page <http://www.tecnotron-software.de/> 
The ';' is misinterpreted I think.

Comment 1

[Jo Hermans]

not a blocker bug

Comment 2

[Andreas Otte]

Seeing this too on windows ... setting component and confirming

Comment 3

[Andreas Otte]

I meant linux ...

Comment 4

[Andreas Otte]

The page contains this redirection:

<meta http-equiv="refresh" content="2;URL=hbl.htm; text/html; charset=iso-8859-1">

Comment 5

[Andreas Otte]

This the comment from the code that parses the header
(nsDocShell::SetupRefreshURIFromHeader):

    // Refresh headers are parsed with the following format in mind
    // <META HTTP-EQUIV=REFRESH CONTENT="5; URL=http://uri">
    // By the time we are here, the following is true:
    // header = "REFRESH"
    // content = "5; URL=http://uri" // note the URL attribute is
    // optional, if it is absent, the currently loaded url is used.
    // Also note that the seconds and URL separator can be either
    // a ';' or a ','. The ',' separator should be illegal but CNN
    // is using it.
    // 
    // We need to handle the following strings, where
    //  - X is a set of digits
    //  - URI is either a relative or absolute URI
    //
    // Note that URI should start with "url=" but we allow omission
    //
    // "" || ";" || "," 
    //  empty string. use the currently loaded URI
    //  and refresh immediately.
    // "X" || "X;" || "X,"
    //  Refresh the currently loaded URI in X seconds.
    // "X; URI" || "X, URI"
    //  Refresh using URI as the destination in X seconds.
    // "URI" || "; URI" || ", URI"
    //  Refresh immediately using URI as the destination.
    // 


>>>>>>>>>>>>>>>>>>>>>>>>>>>>

    // Currently, anything immediately following the URI, if
    // seperated by any char in the set "'\"\t\r\n " will be
    // ignored. So "10; url=go.html ; foo=bar" will work,
    // and so will "10; url='go.html'; foo=bar". However,
    // "10; url=go.html; foo=bar" will result in the uri
    // "go.html;" since ';' and ',' are valid uri characters.

<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

    // Note that we need to remove any tokens wrapping the URI.
    // These tokens currently include spaces, double and single
    // quotes.

    // when done, seconds is 0 or the given number of seconds
    //            uriAttrib is empty or the URI specified


See the marked paragraph. This seems to be an evangelism case. Fix the site.

Comment 6

[Darin Fisher]

-> let docshell folks decide what they want to do with this bug.

Comment 7

[Jean-Pierre Melkonian]

bug 109076 ?

Comment 8

[Adam Lock]

This seems like a WONTFIX evangelism issue to me. It's not a dupe of that other bug.

Comment 9

[Adam Lock]

Marking WONTFIX. I believe evangelism is the best solution here.

Comment 10

[Andreas Otte]

Then it should be reopened and handed over to tech evangelism. Should be simple
to fix ... just add one space before the ;. Let tech evangelism decide if it is
worth  the effort.

Comment 11

[Andreas Otte]

reassigning to correct owner ...

Comment 12

[Kai Lahmann (is there, where MNG is)]

move...

Comment 13

[Cyrille VIVION]

[bug248549notfixed]

Comment 14

[Matthias Versen [:Matti]]

owned by domain grabber

Comment 15

[Matthias Versen [:Matti]]

owned by domain grabber