On Mon, Oct 25, 2004 at 02:01:08PM -0700, Kelson Vibber wrote:
Jorge Arellano Cid wrote:
Attached goes the patch, and to the whole list just in case someone else wants to test it ;) --Apply over a clean rc3 tarball
This fixes:
* Stop button not going off after a redirection loop. * Multiple increases of the redirection hop counter (MTU). * A redirection after a redirection-loop inheriting the failure state.
The patch has its debug messages enabled, just in case.
As soon as you say it's OK, I'll remove the debugging messages, pack the 0.8.3 tarball and upload to the site for broader testing until the elusive Wednesday release. :-)
Looks good. Tested against hyperborea.org (where I saw the spurious errors before) and against mosquito.wordpress.org (where it was originally trapped in the redirect loop). On the latter, with cookies disabled it correctly detects the loop and stops, and with cookies enabled it correctly reloads the first page, picking up the new 200 status instead of reusing the old 302 status, and shows the resulting page.
OK, I also tested mosquito... :-) Well, I packed 0.8.3 and uploaded it for testing. It should have been called rc4, but hopefully this is the last one. More on this on my next email.
Thanks for the last-minute fix!
Thanks to you Kelson for the debugging, reporting and testing. Now 0.8.3 is a lot more polished because of this. -- Cheers Jorge.-