rfc:pecl_http

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision		Previous revision
Last revisionBoth sides next revision
rfc:pecl_http [2015/02/07 12:42] – * Removed http\Env RINIT section * Changed namespace from ''http'' to ''Http'' * Fixed some wordings and list formattings mikerfc:pecl_http [2015/03/09 12:43] – declined pecl_http mike
Line 1: Line 1:
 ====== PHP RFC: Add pecl_http to core ====== ====== PHP RFC: Add pecl_http to core ======
-  * Version: 2.3 +  * Version: 2.4 
-  * Date: 2015-02-07+  * Date: 2014-08-19 
 +  * Last-Modified: 2015-02-20
   * Author: Michael Wallner, <mike@php.net>   * Author: Michael Wallner, <mike@php.net>
-  * Status: Discussion+  * Status: Declined
   * First Published at: http://wiki.php.net/rfc/pecl_http   * First Published at: http://wiki.php.net/rfc/pecl_http
  
Line 31: Line 32:
  
 === Test coverage === === Test coverage ===
-The current test suite provides a code coverage of about 90% and is subject to improvement. \\ Coverage resport of v2: http://dev.iworks.at/ext-http/lcov/http/index.html +The current test suite provides a code coverage of about 90% and is subject to improvement. \\ Coverage resport of v2: http://dev.iworks.at/ext-http/lcov/ext/http/index.html 
  
 There’s one test that currently fails for me due to porting the extension to ZE3, because of reference mismatch and a leak in zend_assign_to_variable_ref() as cause. There’s one test that currently fails for me due to porting the extension to ZE3, because of reference mismatch and a leak in zend_assign_to_variable_ref() as cause.
Line 42: Line 43:
  
 ===== The Guts ===== ===== The Guts =====
-A fully merged tree can be inspected here: \\ https://github.com/m6w6/php-src/tree/merge-http+A fully merged tree can be inspected here (based on v2.2, so slightly out of date): \\ https://github.com/m6w6/php-src/tree/merge-http 
 + 
 +An up-to-date (based on v2.3) pecl_http tree for PHP7 can be found here: \\ https://github.com/php/pecl-http-pecl_http/tree/phpng
  
 ==== Documentation ==== ==== Documentation ====
Line 446: Line 449:
 ===== Vote ===== ===== Vote =====
  
-50%+1 combined "Yes" votes needed for acceptance.+Three way "yes/enabled by default", "yes/disabled by default", "no" where 50%+1 combined "Yes" votes are needed for acceptance. 
 + 
 +<doodle title="Add pecl_http to the core?" auth="mike" voteType="single" closed="true"> 
 +   * Yes, enabled by default 
 +   * Yes, disabled by default 
 +   * No 
 +</doodle> 
 + 
 +\\  
 +\\  
 +Additional simple vote on the namespace prefix ("http" or "php\http") disregarding the case. 
 + 
 +<doodle title="Namespace prefix for pecl_http" auth="mike" voteType="single" closed="true"> 
 +   * http 
 +   * php\http 
 +</doodle> 
 + 
 +===== Discussed and changed items ===== 
 + 
 +== Parsing multipart/form-data and a/www-form-url-encoded for any request method == 
 + 
 +This functionality was removed from the proposal. 
 + 
 +== Parsing a/json into $_POST == 
 + 
 +This functionality was removed from the proposal, which removed the ext/json dependency. 
 + 
 +== Translating charsets of http\QueryString == 
 + 
 +This functionality was removed from the proposal, which removed the ext/iconv dependency. 
 + 
 +== Extended hashing methods for ETags of dynamic content == 
 + 
 +This functionality was removed from the proposal, which removed the ext/hash dependency. 
 + 
 +== Splitting up into smaller RFCs == 
 + 
 +It was requested to split this RFC up into more smaller ones, but mainly only, as observed by me, to *not* bring an HTTP client implementation into the default distribution. These requests were not considered further by me, because I think the client gives substantial value to the overall package. 
 + 
 +== Upgrade path for existing pecl_http users == 
 + 
 +A pecl_http integraded into the default distribution would be considered v3. Upcoming v2 releases could take measures to prepare any transition to the PHP7 API. 
 + 
 +== Namespace choice, or the case of the case == 
 + 
 +I consider this issue non-important, because we do not rely on the case for internal classes, so ''http'', ''HTTP'' and ''Http'' only have cosmetic effect. 
 + 
 +There will be an extra vote on whether to prefix the ''http'' namespace with ''php''. Here, I consider ''PHP'', ''php'' and ''Php'' to be equal as well.
  
 ===== Changelog ===== ===== Changelog =====
Line 469: Line 519:
     * Changed namespace from ''http'' to ''Http''     * Changed namespace from ''http'' to ''Http''
     * Fixed some wordings and list formattings     * Fixed some wordings and list formattings
 +  * 2.4
 +    * Added "Discussed and changed items"
rfc/pecl_http.txt · Last modified: 2017/09/22 13:28 by 127.0.0.1

Page Tools