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

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

[PATCH] websockets support for uwsgi protocol

2013-02-20 Thread Roberto De Ioris
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 various websocket libraries and with the api available in uWSGI 1.9. >From 1.9 sources (with nginx pointing to uwsgi port 3031): ./uwsgi -s :3031 -w tes