[FIX]view-source follows refresh header, and shows the new page

VERIFIED FIXED in mozilla1.0

Status

()

Core
Networking
P1
normal
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Michiel van Leeuwen (email: mvl+moz@), Assigned: bz)

Tracking

Trunk
mozilla1.0
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.9+) Gecko/20020313
BuildID:    2002031321

When visiting a page with a refresh: header, view-source also goes to the url in
the header.

Reproducible: Always
Steps to Reproduce:
1. visit http://www.shop-engine.nl/moztest/redir.php
2. within 5 seconds, view the source
3. wait a couple of seconds


Actual Results:  Window with source also show bugzilla.mozilla.org

Expected Results:  Window with source displays the source of the page

The mentioned url contains this php code:
<?
header("Refresh: 5; url=http://bugzilla.mozilla.org");
?>
taking bug
Assignee: new-network-bugs → bzbarsky
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: review
Priority: -- → P1
Summary: view-source follows refresh header, and shows the new page → [FIX]view-source follows refresh header, and shows the new page
Target Milestone: --- → mozilla1.0

Comment 3

16 years ago
Comment on attachment 74154 [details] [diff] [review]
Patch to fix

r=doron
Attachment #74154 - Flags: review+

Comment 4

16 years ago
Comment on attachment 74154 [details] [diff] [review]
Patch to fix

sr=darin
Attachment #74154 - Flags: superreview+

Comment 5

16 years ago
Comment on attachment 74154 [details] [diff] [review]
Patch to fix

a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #74154 - Flags: approval+
Checked in.  Thanks for the bug report and the excellent testcase, Michiel!
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 7

16 years ago
VERIFIED: Mozilla 1.1

So you want it to say:

<html>
<body>
refresh in 5 sec
</body>
</html>
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.