Refine the redirect check condition

phkelley · phkelley · commit 35e0f3c62968 · 2013-03-25T17:59:30.000-04:00
diff --git a/src/transports/http.c b/src/transports/http.c
@@ -295,9 +295,12 @@ static int on_headers_complete(http_parser *parser)
 		}
 	}
 
-	/* Check for a 302 Found (redirect).
+	/* Check for a redirect.
 	 * Right now we only permit a redirect to the same hostname. */
-	if (parser->status_code == 302 &&
+	if ((parser->status_code == 301 ||
+		 parser->status_code == 302 ||
+		 (parser->status_code == 303 && get_verb == s->verb) ||
+		 parser->status_code == 307) &&
 		t->location) {
 
 		if (s->redirect_count >= 7) {

Original file line number	Diff line number	Diff line change
`@@ -295,9 +295,12 @@ static int on_headers_complete(http_parser *parser)`
`295`	`295`	`}`
`296`	`296`	`}`
`297`	`297`
`298`		`- /* Check for a 302 Found (redirect).`
	`298`	`+ /* Check for a redirect.`
`299`	`299`	`* Right now we only permit a redirect to the same hostname. */`
`300`		`- if (parser->status_code == 302 &&`
	`300`	`+ if ((parser->status_code == 301 \|\|`
	`301`	`+ parser->status_code == 302 \|\|`
	`302`	`+ (parser->status_code == 303 && get_verb == s->verb) \|\|`
	`303`	`+ parser->status_code == 307) &&`
`301`	`304`	`t->location) {`
`302`	`305`
`303`	`306`	`if (s->redirect_count >= 7) {`