com php-src: Add E_DEPRECATED when populating $HTTP _RAW_POST_DATA: main/php_content_types.c main /php_variables.c tests/basic/024.phpt tests/bas
ic/026.phpt
| From: | Michael Wallner | Date: | Mon, 30 Dec 2013 13:28:43 +0000 |
| Subject: | com php-src: Add E_DEPRECATED when populating $HTTP _RAW_POST_DATA: main/php_content_types.c main /php_variables.c tests/basic/024.phpt tests/bas ic/026.phpt |
||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-75512@lists.php.net to get a copy of this message | ||
Commit: 4797f7ad16bc65bda9dca10232c8e0c62a6830ea
Author: Michael Wallner <mike@php.net> Tue, 10 Dec 2013 15:35:41 +0100
Parents: 729f85fad797e31bec1188441049fe525aa78f70
Branches: PHP-5.6 master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=4797f7ad16bc65bda9dca10232c8e0c62a6830ea
Log:
Add E_DEPRECATED when populating $HTTP_RAW_POST_DATA
Changed paths:
M main/php_content_types.c
M main/php_variables.c
M tests/basic/024.phpt
M tests/basic/026.phpt
Diff:
diff --git a/main/php_content_types.c b/main/php_content_types.c
index d15f6a7..ebc879b 100644
--- a/main/php_content_types..c
+++ b/main/php_content_types.c
@@ -65,6 +65,12 @@ SAPI_API SAPI_POST_READER_FUNC(php_default_post_reader)
php_stream_rewind(SG(request_info).request_body);
SET_VAR_STRINGL("HTTP_RAW_POST_DATA", data, length);
+
+ sapi_module.sapi_error(E_DEPRECATED,
+ "Automatically populating $HTTP_RAW_POST_DATA is deprecated and "
+ "will be removed in a future version. To avoid this warning set "
+ "'always_populate_raw_post_data' to '-1' in php.ini and use the "
+ "php://input stream instead.");
}
}
}
diff --git a/main/php_variables.c b/main/php_variables.c
index ab9aee3..9e26756 100644
--- a/main/php_variables.c
+++ b/main/php_variables.c
@@ -734,7 +734,6 @@ static zend_bool php_auto_globals_create_post(const char *name, uint name_len TS
if (PG(variables_order) &&
(strchr(PG(variables_order),'P') || strchr(PG(variables_order),'p'))
&&
- !SG(headers_sent) &&
SG(request_info).request_method &&
!strcasecmp(SG(request_info).request_method, "POST")) {
sapi_module.treat_data(PARSE_POST, NULL, NULL TSRMLS_CC);
diff --git a/tests/basic/024.phpt b/tests/basic/024.phpt
index bf8a206..2e046c0 100644
--- a/tests/basic/024.phpt
+++ b/tests/basic/024.phpt
@@ -10,6 +10,9 @@ a=ABC&y=XYZ&c[]=1&c[]=2&c[a]=3
var_dump($_POST, $HTTP_RAW_POST_DATA);
?>
--EXPECT--
+Deprecated: Automatically populating $HTTP_RAW_POST_DATA is deprecated and will be removed in a
future version. To avoid this warning set 'always_populate_raw_post_data' to
'-1' in php.ini and use the php://input stream instead. in Unknown on line 0
+
+Warning: Cannot modify header information - headers already sent in Unknown on line 0
array(3) {
["a"]=>
string(3) "ABC"
diff --git a/tests/basic/026.phpt b/tests/basic/026.phpt
index b98a31f..a3e34f4 100644
--- a/tests/basic/026.phpt
+++ b/tests/basic/026.phpt
@@ -10,6 +10,9 @@ a=1&b=ZYX
var_dump($_POST, $HTTP_RAW_POST_DATA);
?>
--EXPECT--
+Deprecated: Automatically populating $HTTP_RAW_POST_DATA is deprecated and will be removed in a
future version. To avoid this warning set 'always_populate_raw_post_data' to
'-1' in php.ini and use the php://input stream instead. in Unknown on line 0
+
+Warning: Cannot modify header information - headers already sent in Unknown on line 0
array(0) {
}
string(9) "a=1&b=ZYX"