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
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 [2017/09/22 13:28] (current) – external edit 127.0.0.1
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.1423312966.txt.gz · Last modified: 2017/09/22 13:28 (external edit)