dovecot-2.2: lib-http: Removed more code duplication

dovecot at dovecot.org dovecot at dovecot.org
Sat Aug 10 16:31:39 EEST 2013


details:   http://hg.dovecot.org/dovecot-2.2/rev/1f6059b5c774
changeset: 16685:1f6059b5c774
user:      Timo Sirainen <tss at iki.fi>
date:      Sat Aug 10 16:31:27 2013 +0300
description:
lib-http: Removed more code duplication

diffstat:

 src/lib-http/http-client-request.c |  20 --------------------
 1 files changed, 0 insertions(+), 20 deletions(-)

diffs (30 lines):

diff -r 7fe7cb56b4a6 -r 1f6059b5c774 src/lib-http/http-client-request.c
--- a/src/lib-http/http-client-request.c	Sat Aug 10 15:00:36 2013 +0300
+++ b/src/lib-http/http-client-request.c	Sat Aug 10 16:31:27 2013 +0300
@@ -627,26 +627,6 @@
 		req->payload_offset = 0;
 	}
 
-	/* https://tools.ietf.org/html/draft-ietf-httpbis-p2-semantics-21
-	      Section-7.4.4
-	
-	   -> A 303 `See Other' redirect status response is handled a bit differently.
-	   Basically, the response content is located elsewhere, but the original
-	   (POST) request is handled already.
-	 */
-	if (status == 303 && strcasecmp(req->method, "HEAD") != 0 &&
-		strcasecmp(req->method, "GET") != 0) {
-		// FIXME: should we provide the means to skip this step? The original
-		// request was already handled at this point.
-		req->method = p_strdup(req->pool, "GET");
-
-		/* drop payload */
-		if (req->payload_input != NULL)
-			i_stream_unref(&req->payload_input);
-		req->payload_size = 0;
-		req->payload_offset = 0;
-	}
-
 	/* resubmit */
 	req->state = HTTP_REQUEST_STATE_NEW;
 	http_client_request_do_submit(req);


More information about the dovecot-cvs mailing list