# [VOTE] [RFC] Slim POST data - Thread: https://externals.io/message/70570 - Original archive: https://news-web.php.net/php.internals/70570 - Started by Mike on 2013-12-10T15:40:52+00:00 - Messages: 8 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #70570 — Mike — 2013-12-10T15:40:52+00:00 URL: https://externals.io/message/70570#70570 I opened the vote on the "Slim POST data" RFC: https://wiki.php.net/rfc/slim_post_data#vote Voting will end Tuesday, December 17th 2013. -- Regards, Mike --- ## #70572 — Christopher Jones — 2013-12-10T17:38:49+00:00 URL: https://externals.io/message/70570#70572 · In reply to: #70570 > I opened the vote on the "Slim POST data" RFC: > https://wiki.php.net/rfc/slim_post_data#vote > > Voting will end Tuesday, December 17th 2013. > Mike, Is this really ready for voting? The RFC doesn't look too complete, IMHO. For example, the section "SAPIs Impacted" just contains the text "Web-SAPIs." Which ones have you identified? How are they affected? The RFC needs to be complete enough that an outsider can understand the implications. Another example: the RFC doesn't distinguish between internal default INI settings and php.ini-* values, not even to mention if they are to be kept in sync. Chris -- christopher.jones@oracle.com http://twitter.com/ghrd Free PHP & Oracle book: http://www.oracle.com/technetwork/topics/php/underground-php-oracle-manual-098250.html --- ## #70573 — Michael Wallner — 2013-12-10T18:24:48+00:00 URL: https://externals.io/message/70570#70573 · In reply to: #70572 On Dec 10, 2013 6:38 PM, "Christopher Jones" wrote: > The RFC doesn't look too complete, IMHO. Well, probably, because I don't even know why I had to write an RFC for this kind of stuff. > For example, the section > "SAPIs Impacted" just contains the text "Web-SAPIs." Which ones have > you identified? How are they affected? None are affected in a direct way. It's just that POST data will probably only exist with web SAPIs or to be more explicit, SAPIs providing a read_post handler. Actually, no web SAPI had to be changed, just the CLI server hack needed attention. > The RFC needs to be complete > enough that an outsider can understand the implications. How about there are not many implications? Did you notice the BC recovering userland code? There's really not much about it. I'm very bad with talking much about nothing... > Another example: the RFC doesn't distinguish between internal default > INI settings and php.ini-* values, not even to mention if they are to > be kept in sync. ### From "Changes to PHP-5.6": * 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/off/whatever: BC behavior * 1/on/yes/true: BC behavior INI-default was not touched, I just changed the *commented* value in php.ini to -1, the value providing future proof behavior. ### From "Backward Incompatible Changes to master": $HTTP_RAW_POST_DATA and always_populate_raw_post_data were removed. Nothing more to add here, or is removed non-obvious? Cheers, Mike, who would have loved to talk about all that nearly *three* months ago. --- ## #70653 — Mike — 2013-12-16T08:55:10+00:00 URL: https://externals.io/message/70570#70653 · In reply to: #70570 > I opened the vote on the "Slim POST data" RFC: > https://wiki.php.net/rfc/slim_post_data#vote > > Voting will end Tuesday, December 17th 2013. > One day left, so keep your yays or nays coming. A big "thank you" to the web team for the (visual) upgrade of the wiki, that is higly appreciated! -- Regards, Mike --- ## #70654 — Tjerk Meesters — 2013-12-16T09:11:42+00:00 URL: https://externals.io/message/70570#70654 · In reply to: #70653 > > > […] > > One day left, so keep your yays or nays coming. > > A big "thank you" to the web team for the (visual) upgrade of the wiki, > that is higly appreciated! > Except that the edit and delete icons are broken ;-) https://wiki.php.net/lib/images/pencil.png https://wiki.php.net/lib/images/del.png > > -- > Regards, > Mike > > > -- > > > > -- -- Tjerk --- ## #70656 — Mike — 2013-12-16T10:01:53+00:00 URL: https://externals.io/message/70570#70656 · In reply to: #70654 > > > > > > > […] > > > One day left, so keep your yays or nays coming. > > A big "thank you" to the web team for the (visual) upgrade of > the wiki, > that is higly appreciated! > > > Except that the edit and delete icons are broken ;-) > > > https://wiki.php.net/lib/images/pencil.png > https://wiki.php.net/lib/images/del.png > Not sure, I can follow... I apparently don't have any 404s browsing around and clicking the edit symbol? -- Regards, Mike --- ## #70661 — Ferenc Kovacs — 2013-12-16T10:56:21+00:00 URL: https://externals.io/message/70570#70661 · In reply to: #70656 > > > […] > > Not sure, I can follow... I apparently don't have any 404s browsing > around and clicking the edit symbol? > > -- > Regards, > Mike > > > -- > > > > it seems Hannes removed those files when upgrading the wiki the other day. I've put back the missing icons: http://git.php.net/?p=web/wiki.git;a=commit;h=5226c7ce6acdc64a4f0f7a16a33d531be08702d3 -- Ferenc Kovács @Tyr43l - http://tyrael.hu --- ## #70721 — Mike — 2013-12-17T19:36:23+00:00 URL: https://externals.io/message/70570#70721 · In reply to: #70661 Hi! I closed the poll a munite ago. https://wiki.php.net/rfc/slim_post_data#vote Result: 17 yay, nil nay. Thank you. -- Regards, Mike