socket-io-sticky-session broken with socket.io 2.4.1

[JohnXLivingston]

I don't understand why, but socket-io-sticky-session is broken with socket.io 2.4.1.

Here are the result of a curl request (I removed cookie values):

$ curl 'http://127.0.0.1:12341/socket.io/?EIO=3&transport=polling&t=NSlLG6p'  -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:85.0) Gecko/20100101 Firefox/85.0' -H 'Accept: */*' -H 'Accept-Language: fr,fr-FR;q=0.8,en-US;q=0.5,en;q=0.3' --compressed -H 'Origin: http://localhost:12340' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Referer: http://localhost:12340' -H 'Cookie:[......] -v
* Expire in 0 ms for 6 (transfer 0x560131cdaec0)
*   Trying 127.0.0.1...
* TCP_NODELAY set
* Expire in 200 ms for 4 (transfer 0x560131cdaec0)
* Connected to 127.0.0.1 (127.0.0.1) port 12341 (#0)
> GET /socket.io/?EIO=3&transport=polling&t=NSlLG6p HTTP/1.1
> Host: 127.0.0.1:12341
> Accept-Encoding: deflate, gzip
> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:85.0) Gecko/20100101 Firefox/85.0
> Accept: */*
> Accept-Language: fr,fr-FR;q=0.8,en-US;q=0.5,en;q=0.3
> Origin: http://localhost:12340
> DNT: 1
> Connection: keep-alive
> Referer: http://localhost:12340/planning
> Cookie: [...]
> 
< HTTP/1.1 400 Bad Request
< Connection: close
< 
* Closing connection 0

[darrachequesne]

@JohnXLivingston there is an issue with Node.js >= 12, related to this commit and which would be fixed by #31.

On a related note, we've just published @socket.io/sticky: https://www.npmjs.com/package/@socket.io/sticky

The routing is based on the session ID, and not on the user IP address.

Repository: https://github.com/darrachequesne/socket.io-sticky

[JohnXLivingston]

Thanks for your reply! That's perfect!

[Mohana-Manikandan]

Hello everyone ! Are there any plans on merging the Pull Request #31 ? We are currently stuck on Node v10 because of it.