61122 2011-05-19 06:06:41 -0700 [Soup] http/tests/navigation/post-307-response.html 2013-01-11 07:22:24 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 88961 P2 Normal --- 1 pnormand webkit-unassigned cdumez danw jussi.kukkonen luiz mrobinson svillar oldest_to_newest 406362 0 pnormand 2011-05-19 06:06:41 -0700 Consistently fails on the 3 GTK bots: --- /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/http/tests/navigation/post-307-response-expected.txt 2011-05-19 05:40:59.039013770 -0700 +++ /home/slave/webkitgtk/gtk-linux-64-debug/build/layout-test-results/http/tests/navigation/post-307-response-actual.txt 2011-05-19 05:40:59.039013770 -0700 @@ -1,7 +1,2 @@ -headers CONTENT_TYPE and CONTENT_LENGTH should be present. -CONTENT_TYPE is present. Its value is: application/x-www-form-urlencoded -CONTENT_LENGTH is present. Its value is: 15 - -POST data should be present. -POST data is present. +FAIL: Timed out waiting for notifyDone to be called Test added in r86693. Seems like we should clear Contents-length and Contents-type of the requests that do not have contents, like Qt does in Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp Will skip the test for now. 406591 1 luiz 2011-05-19 13:02:35 -0700 In the 307 specific case the contents of the previous post request should be sent for the new url unchanged, no header should be removed. I have seen that GTK skips the test http/tests/loading/307-after-303-after-post.html as well. It may be that both are failing due to the same reason. 625093 2 cdumez 2012-05-16 00:46:34 -0700 Same issue on EFL port. 740757 3 jussi.kukkonen 2012-10-12 02:40:23 -0700 For reference I'll point out that the test actually enforces non-compliant behaviour, as far as I can see. 307 on a POST should not be followed without specific user permission. Realistically I think breaking spec might be fine though: it's not like most users are going to understand the implications of POST redirect so asking them is not going to be very useful. So I assume libsoup is doing the right thing here but the right thing might not be what webkit wants? 740925 4 mrobinson 2012-10-12 08:21:49 -0700 Here's the original bug (I think) for reference: https://bugs.webkit.org/show_bug.cgi?id=60440 741958 5 danw 2012-10-15 05:06:35 -0700 (In reply to comment #3) > For reference I'll point out that the test actually enforces non-compliant behaviour, as far as I can see. 307 on a POST should not be followed without specific user permission. Realistically I think breaking spec might be fine though: it's not like most users are going to understand the implications of POST redirect so asking them is not going to be very useful. > > So I assume libsoup is doing the right thing here but the right thing might not be what webkit wants? Right. Essentially, WebKit is deciding that, in the context of browsing the web, automatically reposting on 307 is the behavior the user wants/expects, but libsoup, being more general-purpose, isn't going to do it because, eg, maybe it wouldn't be right when doing XMLRPC. I think there's discussion of this somewhere on some other bug, but basically ResourceHandleSoup needs to notice this case itself in closeCallback() (I think?), and send a second request itself in that case. 805256 6 danw 2013-01-11 07:22:24 -0800 fixed by the redirection-handling changes in bug 88961 *** This bug has been marked as a duplicate of bug 88961 ***