Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Maxim Dounin
Hello! On Wed, Feb 20, 2013 at 04:20:29PM +0100, Roberto De Ioris wrote: > > > > > Ok, so the next question is: any specific reason to exclude normal > > CGI responses with "Status" as in your patch? > > > > I in fact don't like the idea of supporting http-like answers with > > status like from

[nginx] svn commit: r5079 - trunk/src/http/modules

2013-02-20 Thread mdounin
Author: mdounin Date: 2013-02-20 16:41:05 + (Wed, 20 Feb 2013) New Revision: 5079 URL: http://trac.nginx.org/nginx/changeset/5079/nginx Log: Connection upgrade support in uwsgi and scgi modules. Prodded by Roberto De Ioris. Modified: trunk/src/http/modules/ngx_http_scgi_module.c trunk

Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Roberto De Ioris
> > Ok, so the next question is: any specific reason to exclude normal > CGI responses with "Status" as in your patch? > > I in fact don't like the idea of supporting http-like answers with > status like from CGI-like protocols, correct way is to use > "Status" header. Not sure why Manlio introdu

Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Maxim Dounin
Hello! On Wed, Feb 20, 2013 at 03:07:53PM +0100, Roberto De Ioris wrote: > > > > >> Hello! > >> > >> (Cc'd nginx-devel@ as this is better list for this discussion.) > >> > >> On Wed, Feb 20, 2013 at 02:30:38PM +0100, Roberto De Ioris wrote: > >> > >>> Hi, the (tiny) attached patch enable support

Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Roberto De Ioris
> >> Hello! >> >> (Cc'd nginx-devel@ as this is better list for this discussion.) >> >> On Wed, Feb 20, 2013 at 02:30:38PM +0100, Roberto De Ioris wrote: >> >>> Hi, the (tiny) attached patch enable support for new websockets >>> handling >>> when the uwsgi protocol is used instead of HTTP. >>> >>>

Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Roberto De Ioris
> Hello! > > (Cc'd nginx-devel@ as this is better list for this discussion.) > > On Wed, Feb 20, 2013 at 02:30:38PM +0100, Roberto De Ioris wrote: > >> Hi, the (tiny) attached patch enable support for new websockets handling >> when the uwsgi protocol is used instead of HTTP. >> >> I have tested i

Re: [PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Maxim Dounin
Hello! (Cc'd nginx-devel@ as this is better list for this discussion.) On Wed, Feb 20, 2013 at 02:30:38PM +0100, Roberto De Ioris wrote: > Hi, the (tiny) attached patch enable support for new websockets handling > when the uwsgi protocol is used instead of HTTP. > > I have tested it with variou