Closed Bug 157861 Opened 22 years ago Closed 22 years ago

have to hit Back twice to return to '300: multiple choices'

Categories

(Core :: Networking: HTTP, defect, P3)

x86
Windows XP
defect

Tracking

()

VERIFIED FIXED
mozilla1.1beta

People

(Reporter: jruderman, Assigned: darin.moz)

References

()

Details

(Whiteboard: [http/1.1 compliance])

Attachments

(1 file)

Steps to reproduce:
1. Load http://www.tentakles.com/TGPs/2002-07-09/picture.html (contains links to
porn).
2. Click a link, or if you want to avoid loading porn, click the home button.
3. Press back.

Result: Nothing happens.  Have to press back a second time.

In addition, if I do this repeatedly:
1. Open a new browser window (Ctrl+N).
2. Paste in http://www.tentakles.com/TGPs/2002-07-09/picture.html and press enter.
3. Close the window.

Result: Step 2 works exactly every other time.

By the way, this is a cool porn site.  Most porn sites make their 404 pages
spammy and it can be hard to tell that the pages are 404s.  This one keeps its
404s clean and offers 300 when you mistype or misguess a character, which is
better than what most non-porn sites do.  Even useit.com doesn't offer automatic
suggestions for mistyped URLs.  What cool feature will the http/1.1 guys think
of next?
Attached patch v1 patchSplinter Review
trivial problem... we were trying to process the redirect, but since the 300
response came without a Location header, the redirect failed.  as a result we
blew away the cache entry for the 300 response.  then on the following attempt
to load the page, we were loading from scratch and it worked.  this patch
simply adds a check to ensure that the cached response contains a Location
header before attempting a cached redirect.
Severity: minor → normal
Status: NEW → ASSIGNED
Keywords: 4xp
Priority: -- → P3
Whiteboard: [http/1.1 compliance]
Target Milestone: --- → mozilla1.2alpha
Comment on attachment 92426 [details] [diff] [review]
v1 patch

r=dougt
Attachment #92426 - Flags: review+
will try to land this for 1.1
Target Milestone: mozilla1.2alpha → mozilla1.1beta
Comment on attachment 92426 [details] [diff] [review]
v1 patch

a=asa (on behalf of drivers) for checkin to 1.1
Attachment #92426 - Flags: approval+
fixed-on-trunk
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Build: 2002-10-22-08-trunk

Back button works.  Can open page all the time.  Marking Verified!
Status: RESOLVED → VERIFIED
QA Contact: tever → jimmylee
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: