Re: http2 broken

2018-07-11 Thread 洪志道
Get it, I'll try it, thank you again. On Thu, Jul 12, 2018 at 1:02 AM Maxim Dounin wrote: > Hello! > > On Wed, Jul 11, 2018 at 08:56:56PM +0800, 洪志道 wrote: > > > But there are still some clients use the old version. > > Is it a way to be compatible with it? > > Or other suggestions? > > @Maxim

Re: http2 broken

2018-07-11 Thread Maxim Dounin
Hello! On Wed, Jul 11, 2018 at 08:56:56PM +0800, 洪志道 wrote: > But there are still some clients use the old version. > Is it a way to be compatible with it? > Or other suggestions? > @Maxim Dounin As I already wrote in ticket #1397, we can try introducing a workaround in nginx, but I'm not

Re: [PATCH] Core: remove unused FIOASYNC.

2018-07-11 Thread Ruslan Ermilov
On Sun, Jul 08, 2018 at 11:36:45PM -0700, Ian Gudger wrote: > Any update on this? The original idea with SIGIO is to wake up master process from sigsuspend() when the message channel is ready for I/O. This feature is currently de-facto unused because if message writing fails, the message is lost

Re: http2 broken

2018-07-11 Thread 洪志道
Thank you. But there are still some clients use the old version. Is it a way to be compatible with it? Or other suggestions? @Maxim Dounin On Wed, Jul 11, 2018 at 5:43 PM tokers wrote: > Hello! > > There is a known issue caused by some obsolete client like old version > okhttp (incomplete

Re: http2 broken

2018-07-11 Thread tokers
Hello! There is a known issue caused by some obsolete client like old version okhttp (incomplete protocol implements). Here is the ticket: https://trac.nginx.org/nginx/ticket/1397. ___ nginx-devel mailing list nginx-devel@nginx.org

Re: http2 broken

2018-07-11 Thread 洪志道
BTW. It works well by the temporary fix. --- a/src/http/v2/ngx_http_v2_filter_module.c +++ b/src/http/v2/ngx_http_v2_filter_module.c @@ -446,7 +446,6 @@ ngx_http_v2_header_filter(ngx_http_request_t *r) if (h2c->table_update) { ngx_log_debug0(NGX_LOG_DEBUG_HTTP, fc->log, 0,

http2 broken

2018-07-11 Thread 洪志道
Hi. The client can't load the image served by nginx-1.13.6+. [App with Android 7.1.2 4.4.4 4.12 series] It works well in nginx-1.13.5, but go wrong in ngnx-1.13.6+. The change is http://hg.nginx.org/nginx/rev/12cadc4669a7. I found the similar problem.