Closed Bug 130931 Opened 22 years ago Closed 22 years ago

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

Categories

(Core :: Networking, defect, P1)

Product:
Core
Component:
Networking
Platform:
x86
Linux
Type:
defect

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla1.0

People

(Reporter: mvl, Assigned: bzbarsky)

References

(
URL
)

Details

Attachments

(1 file)

Patch to fix
926 bytes, patch
doronr
: review+
darin.moz
: superreview+
asa
: approval+
Details | Diff | Splinter Review 
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");
?>
Attached patch Patch to fix — — Splinter Review 

    
    

    
  
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 on attachment 74154 [details] [diff] [review]
Patch to fix

r=doron

        Attachment #74154 -
        Flags: review+

    
    

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

sr=darin

        Attachment #74154 -
        Flags: superreview+

    
    

    
  
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
Closed: 22 years ago
Resolution: --- → FIXED

    
    

    
  
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.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: