1090866 - SPDY SSL websockets unable to connect

Status: RESOLVED FIXED

(Core :: Networking: WebSockets, defect)

Description

[marc]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.111 Safari/537.36

Steps to reproduce:

Set Auto Proxy config with a SPDY proxy, e.g.

function FindProxyForURL(url, host) {return "HTTPS spdy.example.com:443"; }

Visit http://websocketstest.com/

Let me know if you need a spdy server to test with.


Actual results:

WebSockets (Port 443, SSL) fails to connect


Expected results:

WebSockets (Port 443, SSL) should connect

Comment 1

[Patrick McManus [:mcmanus]]

does the proxy allow connect tunnels? It will need to for websockets

Comment 2

[marc]

It should but I did not configure the proxy myself, so I cannot personally confirm. However I have tested it against two spdy server implementations; https://github.com/tatsuhiro-t/spdylay and https://github.com/igrigorik/node-spdyproxy and I get the same result for both.

Comment 3

[Patrick McManus [:mcmanus]]

hmmm.. I can confirm.

but if I change the pac to only use the proxy for the failing case (i.e. only for wss://), then it passes.

I'll look into it.

Comment 4

[Patrick McManus [:mcmanus]]

fyi, we don't actually do spdy/h2 proxying of websockets. The reasons are internal esoteric ones having to do with how websockets takes direct control of the socket layer, but that's a case where we only proxy over h1. (its actually the only case I believe).

however we do proxy over https with h1 - so your config should work as long as your server does https with h1 and is setup to allow connect tunnels.. and node-spdyproxy does indeed do that despite its name.

Comment 5

[Patrick McManus [:mcmanus]]

Attachment: wss over https proxy problem

Comment 6

[Patrick McManus [:mcmanus]]

https://tbpl.mozilla.org/?tree=Try&rev=4aa3b6f555c4

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4c7b57acad96

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4c7b57acad96