From 4c187043c5aac57f354ebb96cc6ff3263411e98d Mon Sep 17 00:00:00 2001 From: Kamil Dudka Date: Tue, 30 Jul 2019 12:59:35 +0200 Subject: http_negotiate: improve handling of gss_init_sec_context() failures If HTTPAUTH_GSSNEGOTIATE was used for a POST request and gss_init_sec_context() failed, the POST request was sent with empty body. This commit also restores the original behavior of `curl --fail --negotiate`, which was changed by commit 6c6035532383e300c712e4c1cd9fdd749ed5cf59. Add regression tests 2077 and 2078 to cover this. Fixes #3992 Closes #4171 --- tests/data/test2078 | 54 +++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 54 insertions(+) create mode 100644 tests/data/test2078 (limited to 'tests/data/test2078') diff --git a/tests/data/test2078 b/tests/data/test2078 new file mode 100644 index 000000000..99bc2dbee --- /dev/null +++ b/tests/data/test2078 @@ -0,0 +1,54 @@ + + + +HTTP +HTTP GET +GSS-API + + + +# Server-side + + +HTTP/1.1 200 OK swsclose +Content-Length: 23 + +This IS the real page! + + + +# Client-side + + +http + + +GSS-API + + +curl --negotiate should not send empty POST request only + + +http://%HOSTIP:%HTTPPORT/2078 -u : --negotiate --data name=value + + + +# Verify data after the test has been "shot" + + +0 + + +^User-Agent:.* + + +POST /2078 HTTP/1.1 +Host: 127.0.0.1:8990 +Accept: */* +Content-Length: 10 +Content-Type: application/x-www-form-urlencoded + +name=value + + + -- cgit v1.2.3