rfc:pecl_http
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext 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 mike | rfc: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, < | * Author: Michael Wallner, < | ||
- | * Status: | + | * Status: |
* First Published at: http:// | * First Published at: 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:// | + | The current test suite provides a code coverage of about 90% and is subject to improvement. \\ Coverage resport of v2: http:// |
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:// | + | A fully merged tree can be inspected here (based on v2.2, so slightly out of date): \\ https:// |
+ | |||
+ | An up-to-date (based on v2.3) pecl_http tree for PHP7 can be found here: \\ https:// | ||
==== Documentation ==== | ==== Documentation ==== | ||
Line 446: | Line 449: | ||
===== Vote ===== | ===== Vote ===== | ||
- | 50%+1 combined " | + | Three way " |
+ | |||
+ | <doodle title=" | ||
+ | * Yes, enabled by default | ||
+ | * Yes, disabled by default | ||
+ | * No | ||
+ | </ | ||
+ | |||
+ | \\ | ||
+ | \\ | ||
+ | Additional simple vote on the namespace prefix (" | ||
+ | |||
+ | <doodle title=" | ||
+ | * http | ||
+ | * php\http | ||
+ | </ | ||
+ | |||
+ | ===== Discussed and changed items ===== | ||
+ | |||
+ | == Parsing multipart/ | ||
+ | |||
+ | 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, | ||
+ | |||
+ | There will be an extra vote on whether to prefix the '' | ||
===== Changelog ===== | ===== Changelog ===== | ||
Line 469: | Line 519: | ||
* Changed namespace from '' | * Changed namespace from '' | ||
* Fixed some wordings and list formattings | * Fixed some wordings and list formattings | ||
+ | * 2.4 | ||
+ | * Added " |
rfc/pecl_http.1423312966.txt.gz · Last modified: 2017/09/22 13:28 (external edit)