X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=src%2Fphp_http_info.c;h=81bf727b1e4d135bf931af3b084984743f41dc0a;hp=4fb067fcd2610e63a4110663e3172a3e62d9ca64;hb=76b679d046ac10c00eb13da217f2adaaad580251;hpb=bdd6edb59194cda9e5fcb393c48ab4230fceb32a

diff --git a/src/php_http_info.c b/src/php_http_info.c
index 4fb067f..81bf727 100644
--- a/src/php_http_info.c
+++ b/src/php_http_info.c
@@ -49,9 +49,49 @@ void php_http_info_free(php_http_info_t **i)
 	}
 }
 
+void php_http_info_to_string(php_http_info_t *info, char **str, size_t *len, const char *eol TSRMLS_DC)
+{
+	char *tmp = NULL;
+
+	if (info->http.version.major == 2) {
+		if (info->type == PHP_HTTP_REQUEST) {
+			*len = spprintf(str, 0, "%s %s HTTP/2%s",
+					info->http.info.request.method?info->http.info.request.method:"UNKNOWN",
+					info->http.info.request.method&&!strcasecmp(info->http.info.request.method,"CONNECT")?(
+					info->http.info.request.url?php_http_url_authority_to_string(info->http.info.request.url, &(tmp), NULL):"0"):(
+					info->http.info.request.url?php_http_url_to_string(info->http.info.request.url, &(tmp), NULL, 0):"/"),
+					eol);
+		} else if (info->type == PHP_HTTP_RESPONSE) {
+			*len = spprintf(str, 0, "HTTP/2 %d%s%s%s",
+					info->http.info.response.code?info->http.info.response.code:200,
+					info->http.info.response.status&&*info->http.info.response.status ? " ":"",
+					STR_PTR(info->http.info.response.status),
+					eol);
+		}
+	} else if (info->type == PHP_HTTP_REQUEST) {
+		*len = spprintf(str, 0, "%s %s HTTP/%u.%u%s",
+				info->http.info.request.method?info->http.info.request.method:"UNKNOWN",
+				info->http.info.request.method&&!strcasecmp(info->http.info.request.method,"CONNECT")?(
+				info->http.info.request.url?php_http_url_authority_to_string(info->http.info.request.url, &(tmp), NULL):"0"):(
+				info->http.info.request.url?php_http_url_to_string(info->http.info.request.url, &(tmp), NULL, 0):"/"),
+				info->http.version.major||info->http.version.major?info->http.version.major:1,
+				info->http.version.major||info->http.version.minor?info->http.version.minor:1,
+				eol);
+	} else if (info->type == PHP_HTTP_RESPONSE){
+		*len = spprintf(str, 0, "HTTP/%u.%u %d%s%s%s", info->http.version.major||info->http.version.major?info->http.version.major:1,
+				info->http.version.major||info->http.version.minor?info->http.version.minor:1,
+				info->http.info.response.code?info->http.info.response.code:200,
+				info->http.info.response.status&&*info->http.info.response.status ? " ":"",
+				STR_PTR(info->http.info.response.status),
+				eol);
+	}
+
+	STR_FREE(tmp);
+}
+
 php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_header TSRMLS_DC)
 {
-	const char *end, *http;
+	const char *end, *http, *off;
 	zend_bool free_info = !info;
 	
 	/* sane parameter */
@@ -71,9 +111,21 @@ php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_head
 	
 	info = php_http_info_init(info TSRMLS_CC);
 
-	/* and nothing than SPACE or NUL after HTTP/X.x */
-	if (!php_http_version_parse(&info->http.version, http TSRMLS_CC)
-	||	(http[lenof("HTTP/X.x")] && (!PHP_HTTP_IS_CTYPE(space, http[lenof("HTTP/X.x")])))) {
+	if (!php_http_version_parse(&info->http.version, http TSRMLS_CC)) {
+		if (free_info) {
+			php_http_info_free(&info);
+		}
+		return NULL;
+	}
+
+	/* clumsy fix for changed libcurl behaviour in 7.49.1, see https://github.com/curl/curl/issues/888 */
+	off = &http[lenof("HTTP/X")];
+	if (info->http.version.major < 2 || (info->http.version.major == 2 && *off == '.')) {
+		off += 2;
+	}
+
+	/* and nothing than SPACE or NUL after HTTP/X(.x) */
+	if (*off && (!PHP_HTTP_IS_CTYPE(space, *off))) {
 		if (free_info) {
 			php_http_info_free(&info);
 		}
@@ -90,11 +142,11 @@ php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_head
 
 	/* is response */
 	if (pre_header == http) {
-		const char *status = NULL, *code = http + sizeof("HTTP/X.x");
+		const char *status = NULL, *code = off;
 		
 		info->type = PHP_HTTP_RESPONSE;
 		while (' ' == *code) ++code;
-		if (code && end > code) {
+		if (end > code) {
 			/* rfc7230#3.1.2 The status-code element is a 3-digit integer code */
 			PHP_HTTP_INFO(info).response.code = 100*(*code++ - '0');
 			PHP_HTTP_INFO(info).response.code += 10*(*code++ - '0');
@@ -120,7 +172,7 @@ php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_head
 	}
 	
 	/* is request */
-	else if (*(http - 1) == ' ' && (!http[lenof("HTTP/X.x")] || http[lenof("HTTP/X.x")] == '\r' || http[lenof("HTTP/X.x")] == '\n')) {
+	else if (*(http - 1) == ' ' && (!*off || *off == '\r' || *off == '\n')) {
 		const char *url = strchr(pre_header, ' ');
 		
 		info->type = PHP_HTTP_REQUEST;
@@ -135,9 +187,9 @@ php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_head
 			if (http > url) {
 				/* CONNECT presents an authority only */
 				if (strcasecmp(PHP_HTTP_INFO(info).request.method, "CONNECT")) {
-					PHP_HTTP_INFO(info).request.url = php_http_url_parse(url, http - url, ~0 TSRMLS_CC);
+					PHP_HTTP_INFO(info).request.url = php_http_url_parse(url, http - url, PHP_HTTP_URL_STDFLAGS TSRMLS_CC);
 				} else {
-					PHP_HTTP_INFO(info).request.url = php_http_url_parse_authority(url, http - url, ~0 TSRMLS_CC);
+					PHP_HTTP_INFO(info).request.url = php_http_url_parse_authority(url, http - url, PHP_HTTP_URL_STDFLAGS TSRMLS_CC);
 				}
 				if (!PHP_HTTP_INFO(info).request.url) {
 					PTR_SET(PHP_HTTP_INFO(info).request.method, NULL);
@@ -155,7 +207,7 @@ php_http_info_t *php_http_info_parse(php_http_info_t *info, const char *pre_head
 		return info;
 	}
 
-	/* some darn header containing HTTP/X.x */
+	/* some darn header containing HTTP/X(.x) */
 	else {
 		if (free_info) {
 			php_http_info_free(&info);