Hi Davey, Thanks for your answer. I still have small projects on my pipeline that I want to finish but I'll start looking at libnghttp2 and if I get some time I'll recontact you to maybe help you on remaining tasks :-) Do you have specifics stuffs that you would like me to look at ?
About the patch if there is no objection i'll commit it to the 7.1 branch soon. Pierrick On 23 April 2016 at 20:24, Davey Shafik <da...@php.net> wrote: > Hi Pierrick, > > This should be in master for 7.1, alongside my RFC'ed patch for server > push support. > > You emailed me directly about the aforementioned patch so I'll just > respond here as it's relevant: > > The patch should hit in 7.1 but it has been requested that tests be added > — and we can't add tests with a server push supporting HTTP/2 server > against which to push. > > I'm working with Pierre Joye (CC'ed) to try and update the cli-server to > use libnghttp2 to turn it into a fully fledged HTTP/2 compliant server with > support for server push and things like multiplexing (see: > http://wiki.php.net/rfc/cli_server_http2). Curl also uses libnghttp2 for > it's HTTP/2 support. > > I hope to eventually (7.2+) use libnghttp2 to add an ext/nghttp2 HTTP > client and update the HTTP streams layer to support HTTP/2 also. > > I'd welcome your collaboration on any of this. > > - Davey > > On Sat, Apr 23, 2016 at 12:30 PM, Pierrick Charron <pierr...@adoy.net> > wrote: > >> Hi internals, >> >> I took some time to add some easy to implement new "features" that were >> implemented in libcurl but missing in ext/curl. Most of them are just >> exposing a new constant in ext/curl and dispatched in the curl_setopt >> function. I created a branch over master but the patch is applicable >> without conflict the 7.0 branch. >> >> https://github.com/php/php-src/compare/master...adoy:curl-update >> >> I was wondering since it's not a big patch and it only introduce new >> constants : >> >> - Should I commit this patch to 7.0 and master or only master >> - Should I do a RFC to include this patch ? >> >> Thanks for your feedback >> Pierrick >> >> >> >> >> >> >> >
