Closed Bug 10926 Opened 25 years ago Closed 24 years ago

301/302 Redirects don't update location Bar

Categories

(Core :: Networking, defect, P3)

x86
All
defect

Tracking

()

VERIFIED DUPLICATE of bug 30783

People

(Reporter: solomonr, Assigned: gagan)

References

()

Details

The 7-29-99 build doesn't redirect the browser to the proper page. To recreate yahoo.com. Click on stock quote, the link www.yahoo.com/r/sq goes into the location instead of finance.yahoo.com
Assignee: don → radha
Component: Browser-General → Necko
QA Contact: leger → paulmac
The redirection is working fine, the problem is the Location Bar is not updated with the new URL. It is a 302 redirect. To reproduce, simply go to http://www.yahoo.com/r/sq You will be redirected to http://finance.yahoo.com/?u correctly but http://www.yahoo.com/r/sq is still in the Location Bar. Not sure who gets this, assigning to radha, cc'ing rpotts
Summary: Redirection dosen't work → 301/302 Redirects don't update location Bar
*** Bug 1561 has been marked as a duplicate of this bug. ***
Maybe I don't know what I'm talking about, but wouldn't it be better if 302s showed the source rather than target in the location bar, so if you bookmarked them you'd get the source and could be redirected elsewhere when the 302 target changes? But then again, maybe the solution is to bring up a dialog listing the choices when you've been involved in 302s and try to bookmark the current page.
Or is that a 307. Looking at the HTTP 1.1 RFC, I can't see the diff between 302/307.
Status: NEW → ASSIGNED
Target Milestone: M9
Assignee: radha → warren
Status: ASSIGNED → NEW
nsWebShell doesn't get notified in OnEndDocumentLoad for the redirected url. It only gets notified for the original url and a couple of xul files in navigator/content/default. Reassigning to warren. he fixed up few things in redirection code recently.
Status: NEW → ASSIGNED
Cc'ing Gagan because he's dealing with another location bar updating issue. Currently I believe that we're adding the redirect URL to the same load group as the original, and then removing the original. This has the property of only doing the OnEndDocumentLoad when the redirected URL completes. Maybe we need to swap the order so that the OnEndDocumentLoad fires, and then reuse the group to being another load (although at that point we'd need the doc loader, not just its load group). Rick: Help.
Assignee: warren → rpotts
Status: ASSIGNED → NEW
More docloader madness. Reassigning to Rick since he's going to work with Radha on that.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
using today's build on win98, I can't reproduce this bug, the redirect does update the location bar. Marking fixed
Status: RESOLVED → VERIFIED
Marking Verified.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
actually, this is not working for me on any platform, using M9 or M10 builds, re-opening. Again, the problem is simply the URL bar is not updated on a redirect to the new URL. The actual redirect is working correctly.
Target Milestone: M9 → M10
that being said, it does not seem remotely close to an M9 stopper, so moving to M10, unless someone objects...
OS: Windows 98 → All
The URL bar is failing to update on current Linux builds from the tip as well. Another example of this behavior is http://www.mozilla.org/NPL/ returning a redirect to http://www.mozilla.org/MPL/
Target Milestone: M10 → M11
m11
Assignee: rpotts → gagan
Status: REOPENED → NEW
Gagan, since you're looking at progress/status notifications, you should probably take this one too. I assume there will be some redirect observer interface accessible from the event sink getter that you can call from the http protocol. Then, the docloader will receive it and pass it to the webshell to display.
*** Bug 15857 has been marked as a duplicate of this bug. ***
Status: NEW → ASSIGNED
we dont implement any http event sink as yet... thats why we wont get the redirects. However I think we should really be checking for the uri in the onEndRequest or someplace and use that to update. Taking over but probably is webshell owners(!?!)
*** Bug 17131 has been marked as a duplicate of this bug. ***
Target Milestone: M11 → M12
so should rpotts, travis or buster get this? ->12
Status: ASSIGNED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → DUPLICATE
This really is the same roblem as in 10647. we need to change the webshell to also give us an HTTPEventSink to correctly handle redirects. I guess this should eventually be a factor in the new webshell design (so travis should know) but probably rpotts would be involved in the implementation. *** This bug has been marked as a duplicate of 10647 ***
Status: RESOLVED → VERIFIED
Marking Verified as a dup.
Bulk move of all Necko (to be deleted component) bugs to new Networking component.
reopening, todays linux build exibits this problem. not opening the bug this was duped aginst since i can only see the symptom and not the cause. also note that if you manually enter the URL in the location bar, there is a differant effect. an html page from freshmeat is loaded, then a redirection occurs which works fine. only when the link is followed with a click does it not update the location bar. milestone reset
Status: VERIFIED → REOPENED
Resolution: DUPLICATE → ---
Target Milestone: M12 → ---
this is now just a dup. *** This bug has been marked as a duplicate of 30783 ***
Status: REOPENED → RESOLVED
Closed: 25 years ago24 years ago
Resolution: --- → DUPLICATE
Verified dupe.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.