rfc:slim_post_data
Differences
This shows you the differences between two versions of the page.
Next revision | Previous revision | ||
rfc:slim_post_data [2013/11/19 09:35] – created mike | rfc:slim_post_data [2017/09/22 13:28] (current) – external edit 127.0.0.1 | ||
---|---|---|---|
Line 1: | Line 1: | ||
====== PHP RFC: Slim POST data ====== | ====== PHP RFC: Slim POST data ====== | ||
- | * Version: 1.0 | + | * Version: 1.3 |
- | * Date: 2013-11-19 | + | * Date: 2013-12-17 |
* Author: Michael Wallner < | * Author: Michael Wallner < | ||
- | * Status: | + | * Status: |
* First Published at: https:// | * First Published at: https:// | ||
===== Introduction ===== | ===== Introduction ===== | ||
- | This is not a new feature per-se. | + | A memory usage improvement regarding receiving HTTP payloads with a potential reduction of 200-300%. |
- | + | ||
- | It is a memory usage improvement regarding receiving HTTP payloads with a potential reduction of 200-300%. | + | |
===== Proposal ===== | ===== Proposal ===== | ||
Line 21: | Line 19: | ||
If the request had another request method or an unrecognized content type, the payload is available through the re-usable, just-in-time %%php:// | If the request had another request method or an unrecognized content type, the payload is available through the re-usable, just-in-time %%php:// | ||
- | ===== Backward Incompatible Changes | + | ===== Proposed PHP Version(s) |
- | $HTTP_RAW_POST_DATA and always_populate_raw_post_data were removed. | + | Next 5.x, i.e. 5.6 |
- | BC can be restored with: | + | ===== SAPIs Impacted ===== |
- | <code php> | + | Web-SAPIs. |
- | $GLOBALS[" | + | |
- | </ | + | |
- | However, always_populate_raw_post_data may be re-introduced again to mitigate part of this BC issue. It could even be made PHP_INI_USER to further minimize the impact, but I guess it makes no difference if one has to call ini_set or file_get_contents... | + | ===== Unaffected PHP Functionality ===== |
- | ===== Proposed PHP Version(s) ===== | + | Standard form POSTs and file uploads. |
- | Next 5.x, i.e. 5.6 | + | ===== Changes to PHP-5.6 ===== |
- | ===== SAPIs Impacted ===== | + | * Re-usable, optioanlly JITty initialized %%php:// |
+ | * Change always_populate_raw_post_data INI setting to accept three values instead of two. | ||
+ | * -1: The behavior of master; don't ever populate %%$GLOBALS[HTTP_RAW_POST_DATA]%% | ||
+ | * 0/ | ||
+ | * 1/ | ||
- | Web-SAPIs. | + | ===== Backward Incompatible Changes to master ===== |
+ | |||
+ | $HTTP_RAW_POST_DATA and always_populate_raw_post_data were removed. | ||
+ | |||
+ | BC can be restored with: | ||
+ | |||
+ | <code php> | ||
+ | $GLOBALS[" | ||
+ | </ | ||
===== Impact to Existing Extensions ===== | ===== Impact to Existing Extensions ===== | ||
Line 57: | Line 65: | ||
+ | + | ||
</ | </ | ||
- | |||
- | ===== Open Issues ===== | ||
- | |||
- | * Reintroduce always_populate_raw_post_data? | ||
- | |||
- | ===== Unaffected PHP Functionality ===== | ||
- | |||
- | Standard form POSTs and file uploads. | ||
===== Future Scope ===== | ===== Future Scope ===== | ||
Line 72: | Line 72: | ||
===== Proposed Voting Choices ===== | ===== Proposed Voting Choices ===== | ||
- | * Yes, leave it as is. | + | * Yes |
- | * Yes, but re-introduce always_populate_raw_post_data. | + | * No |
- | * No, go away. | + | |
===== Patches and Tests ===== | ===== Patches and Tests ===== | ||
- | More or less: | + | To see the changes already in master, do, more or less: |
<code shell> | <code shell> | ||
Line 85: | Line 84: | ||
http:// | http:// | ||
+ | |||
+ | Always_populate_raw_post_data patch is available here: | ||
+ | |||
+ | https:// | ||
===== References ===== | ===== References ===== | ||
Line 92: | Line 95: | ||
* http:// | * http:// | ||
+ | ===== Changes ===== | ||
+ | |||
+ | * 1.1 | ||
+ | * Added PHP-5.6 patch to re-introduce always_populate_raw_post_data (-1/0/1) | ||
+ | * 1.2 | ||
+ | * Re-ordered the RFC and reworded some sections. | ||
+ | * 1.3 | ||
+ | * Accepted. | ||
+ | |||
+ | ===== Vote ===== | ||
+ | <doodle title=" | ||
+ | * Yes | ||
+ | * No | ||
+ | </ |
rfc/slim_post_data.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1