tg123 / websockify-nginx-module

Embed websockify into Nginx (convert any tcp connection into websocket)
MIT License
140 stars 59 forks source link

unix socket #7

Closed bAndie91 closed 8 years ago

bAndie91 commented 8 years ago

does it support unix domain sockets?

tg123 commented 8 years ago

upstreams are domain sockets or listen as a domain socket.

bAndie91 commented 8 years ago

upstream UDS, eg. websockify_pass unix:/tmp/vnc.sock; In meantime, I checked my config and I discovered websockify does not parse parameter properly, when it is passed as variable: works: websockify_pass unix:/tmp/vnc.sock; buggy: set $socket unix:/tmp/vnc.sock; websockify_pass $socket; log: [error] 16553#0: *30 no port in upstream "/tmp/vnc.sock" while sending to client

tg123 commented 8 years ago

seems a parsing problem here

https://github.com/tg123/websockify-nginx-module/blob/master/ngx_http_websockify_module.c#L813

I will try to fix it