X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=http_functions.c;h=ee8fba4e987df6ff8b0664a83dfa7095b3012a4f;hp=7612013c070aca40fb1c63f6c5bc14b28b547ea6;hb=3177ba2e5ef642f38c01568afcaa547df65f3e74;hpb=902d195a198f4976c8ff081a95cdd3e315c14f5f

diff --git a/http_functions.c b/http_functions.c
index 7612013..ee8fba4 100644
--- a/http_functions.c
+++ b/http_functions.c
@@ -202,7 +202,7 @@ PHP_FUNCTION(http_send_status)
 		RETURN_FALSE;
 	}
 	if (status < 100 || status > 510) {
-		http_error_ex(E_WARNING, HTTP_E_HEADER, "Invalid HTTP status code (100-510): %d", status);
+		http_error_ex(HE_WARNING, HTTP_E_HEADER, "Invalid HTTP status code (100-510): %d", status);
 		RETURN_FALSE;
 	}
 
@@ -474,7 +474,7 @@ PHP_FUNCTION(http_redirect)
 			array_init(params);
 		}
 		if (add_assoc_string(params, PS(session_name), PS(id), 1) != SUCCESS) {
-			http_error(E_WARNING, HTTP_E_ENCODE, "Could not append session information");
+			http_error(HE_WARNING, HTTP_E_RUNTIME, "Could not append session information");
 		}
 	}
 
@@ -497,8 +497,10 @@ PHP_FUNCTION(http_redirect)
 	}
 	efree(URI);
 
-	if ((SUCCESS == http_send_header(LOC)) && (SUCCESS == http_send_status((permanent ? 301 : 302)))) {
-		php_body_write(RED, strlen(RED) TSRMLS_CC);
+	if ((SUCCESS == http_send_header_string(LOC)) && (SUCCESS == http_send_status((permanent ? 301 : 302)))) {
+		if (SG(request_info).request_method && strcmp(SG(request_info).request_method, "HEAD")) {
+			PHPWRITE(RED, strlen(RED));
+		}
 		RETURN_TRUE;
 	}
 	RETURN_FALSE;
@@ -618,7 +620,6 @@ PHP_FUNCTION(http_split_response)
 	array_init(zheaders);
 
 	if (SUCCESS != http_split_response(response, response_len, Z_ARRVAL_P(zheaders), &body, &body_len)) {
-		http_error(E_WARNING, HTTP_E_PARSE, "Could not parse HTTP response");
 		RETURN_FALSE;
 	}
 
@@ -642,7 +643,6 @@ PHP_FUNCTION(http_parse_headers)
 
 	array_init(return_value);
 	if (SUCCESS != http_parse_headers(header, return_value)) {
-		http_error(E_WARNING, HTTP_E_PARSE, "Could not parse HTTP headers");
 		zval_dtor(return_value);
 		RETURN_FALSE;
 	}
@@ -674,7 +674,7 @@ PHP_FUNCTION(http_get_request_body)
 	NO_ARGS;
 
 	if (SUCCESS == http_get_request_body(&body, &length)) {
-		RETURN_STRING(body, (int) length, 0);
+		RETURN_STRINGL(body, (int) length, 0);
 	} else {
 		RETURN_NULL();
 	}
@@ -689,7 +689,7 @@ PHP_FUNCTION(http_match_request_header)
 {
 	char *header, *value;
 	int header_len, value_len;
-	zend_bool match_case = 0, result = 0;
+	zend_bool match_case = 0;
 
 	if (SUCCESS != zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss|b", &header, &header_len, &value, &value_len, &match_case)) {
 		RETURN_FALSE;
@@ -737,7 +737,6 @@ PHP_FUNCTION(http_match_request_header)
  *  - timeout:          int, seconds the request may take
  *  - connecttimeout:   int, seconds the connect may take
  *  - onprogress:       mixed, progress callback
- *  - ondebug:          mixed, debug callback
  * </pre>
  *
  * The optional third parameter will be filled with some additional information
@@ -1228,7 +1227,7 @@ PHP_FUNCTION(http_build_query)
 	}
 
 	if (Z_TYPE_P(formdata) != IS_ARRAY && Z_TYPE_P(formdata) != IS_OBJECT) {
-		http_error(E_WARNING, HTTP_E_PARAM, "Parameter 1 expected to be Array or Object.  Incorrect value given.");
+		http_error(HE_WARNING, HTTP_E_INVALID_PARAM, "Parameter 1 expected to be Array or Object.  Incorrect value given.");
 		RETURN_FALSE;
 	}
 
@@ -1255,7 +1254,7 @@ PHP_FUNCTION(http_build_query)
 
 PHP_FUNCTION(http_test)
 {
-	RETURN_NULL();
+	RETURN_BOOL(HTTP_G(only_exceptions));
 }
 
 /*
@@ -1266,3 +1265,4 @@ PHP_FUNCTION(http_test)
  * vim600: noet sw=4 ts=4 fdm=marker
  * vim<600: noet sw=4 ts=4
  */
+