Re: [PATCH 12 of 14] Proxy: add HTTP/2 support

2017-08-08 Thread Maxim Dounin
Hello! On Tue, Jul 25, 2017 at 06:28:16PM -0700, Piotr Sikora wrote: > Hey Maxim, > > > There are serious concerns about this and with other patches. > > Most generic ones are outlined below. > > > > 1. Server-side HTTP/2 code complexity concerns. > > > > As per discussion with Valentin,

Re: [PATCH 12 of 14] Proxy: add HTTP/2 support

2017-07-31 Thread Piotr Sikora via nginx-devel
Hey Maxim, On Tue, Jul 25, 2017 at 6:28 PM, Piotr Sikora wrote: > Hey Maxim, > >> There are serious concerns about this and with other patches. >> Most generic ones are outlined below. >> >> 1. Server-side HTTP/2 code complexity concerns. >> >> As per discussion with

Re: [PATCH 12 of 14] Proxy: add HTTP/2 support

2017-07-25 Thread Piotr Sikora via nginx-devel
Hey Maxim, > There are serious concerns about this and with other patches. > Most generic ones are outlined below. > > 1. Server-side HTTP/2 code complexity concerns. > > As per discussion with Valentin, introducing client-related code > paths in the server HTTP/2 code seems to complicate things.