[Webkit-unassigned] [Bug 70683] [Qt] Change HTTP redirection behavior to make Firefox/Chromium

Mon Oct 24 06:38:24 PDT 2011

https://bugs.webkit.org/show_bug.cgi?id=70683

Dawit A. <adawit at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[Qt] HTTP redirection is    |[Qt] Change HTTP
                   |not handled correctly for a |redirection behavior to
                   |303 response                |make Firefox/Chromium

--- Comment #7 from Dawit A. <adawit at kde.org>  2011-10-24 06:38:24 PST ---
(In reply to comment #6)
> (From update of attachment 112115 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=112115&action=review
> 
> Generally I would prefer this approach to the other one. But that's just MHO, I'm not a real expert on this particular topic.

So do I and I do not think any particular exerptise is required for addressing this particular problem. The RFC on the topic, RFC 2616 section 10.3, is clear as a day light and so is the violation by the other browsers. One would only have to vist the wonderful test site mentioned in comment #5 with QtTestBrowser, Firefox and Chromium to see the differences in behavior. I think addressing that should be a no brainer since QtWebKit should be as close to these widely deployed browsers as possible.

Since we seem to agree this is the best approach, I will change this bug's title to match this patch.

> > Source/WebCore/ChangeLog:13
> > +        No new tests required as existing redirection tests should cover this
> > +        scenario as well.
> 
> Hm, this _is_ touchy code. I think that this patch should add a (http) layout test to cover this. If there exists a test already, then it should be unskipped.

There is already a layout test that was added due to another bug I reported a while back and I do not think it is skipped. See 

https://bugs.webkit.org/show_bug.cgi?id=60440#c24

However, I am unsure if the tests cover all the HTTP operations or just POST.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.