Trivial: Websocket event handler janus is not properly handling the path part in the backend parameter

[JanFellner]

Humbled over a trivial bug (weekend exercise ;)) where janus seems not to properly handle the url path parameter in the janus.eventhandler.wsevh.jcfg

Path was configured like this:
backend = "ws://10.20.3.220:3000/janus/ws"

But the request is missing the first slash and therefore not properly routed through a straight forward node-express-router implementation.

GET janus/ws HTTP/1.1
Pragma: no-cache
Cache-Control: no-cache
Host: 10.20.3.220
Origin: http://10.20.3.220
Upgrade: websocket
Connection: Upgrade
Sec-WebSocket-Key: xvZWqCblY08tyMEfCV2Orw==
Sec-WebSocket-Protocol: janus-event-handlers
Sec-WebSocket-Version: 13

HTTP/1.1 400 Bad Request
Connection: close

I could only solve it with a double slash
backend = "ws://10.20.3.220:3000//janus/ws"

[tmatth]

@JanFellner sounds like this issue?
warmcat/libwebsockets#414

See their updated usage here:
warmcat/libwebsockets@d2c140c

[lminiero]

@tmatth thanks for the pointer, so this means we'll have to fix the URI parsing in our code, correct?

[lminiero]

Should be fixed now.

[JanFellner]

Will check thanks... :)

[JanFellner]

Works as expected. Thanks.