[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [libmicrohttpd] Tiny request

From: Christian Grothoff
Subject: Re: [libmicrohttpd] Tiny request
Date: Thu, 04 Dec 2014 02:00:46 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.2.0

Dear Greg,

Yes, full support for WebSockets is more complex and an open feature
request for a while ( I would
like to do this "right", but other things are keeping me busy.

I thought your patch's goal was merely to enable a slightly more
"compliant" failure situtation for WebSocket clients.  If Intel has a
strong need for 'real' WebSocket support in MHD, we should discuss this
off-list, as this not a 'tiny request' anymore ;-).

Happy hacking!


On 12/04/2014 01:39 AM, Junker, Gregory wrote:
> Thanks Christian
> Unfortunately, I realized that is not actually enough. WebSocket uses
> the existing TCP connection with the webserver for further
> WebSocket-framed traffic. This requires that MHD "orphan" the
> MHD_Connection it originally set up for the incoming request (and
> release the existing socket descriptor to ownership and control of
> user code). I've been looking through the source in daemon.c and it
> looks like a fairly complex (various combinations of
> polling/not-polling/selecting/thread-or-not-thread-per-connection,
> and so on). Am I just overreading the amount of work, or is it
> actually non-trivial to pull this off?
> Thanks again! Greg
> -----Original Message----- From:
> address@hidden
> [mailto:address@hidden On
> Behalf Of Christian Grothoff Sent: Wednesday, December 3, 2014 3:47
> PM To: address@hidden Subject: Re: [libmicrohttpd] Tiny
> request
> Done (SVN 34474). -Christian
> On 12/04/2014 12:30 AM, Junker, Gregory wrote:
>> Hi all
>> I need a tiny addition made to connection.c.
>> In keepalive_possible(), can we change the line that says
>> if (0 == strcasecmp (end, "close"))
>> to
>> if (0 == strcasecmp (end, "close") || 0 == strcasecmp (end,
>> "upgrade"))
>> ?
>> This would make it possible to use libmicrohttpd in a WebSocket
>> (RFC6455) environment. Currently, the way that the WebSocket
>> protocol works, it sends "Connection: Upgrade" in the headers with
>> the initial handshake, which causes this check to fail and
>> (ultimately) insert "Connection: Keep-Alive" in the response
>> headers (which cause any compliant WebSocket client to fail the
>> handshake, since it needs "Connection: Upgrade" in the response
>> headers, and there is no way to remove this keepalive header from
>> the response from outside of MHD, as it is added automatically to
>> the response buffer).
>> Note that this is only an HTTP/1.1 issue, AFAIK (I am pretty sure,
>> though not positive, that WebSocket is not compatible with
>> HTTP/1.0).
>> Thanks! Greg

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]