Skip to content
This repository has been archived by the owner on Aug 25, 2022. It is now read-only.

Authorization and handshaking #37

Closed
wants to merge 1 commit into from

Conversation

BruceLampson
Copy link

Elephant.io rocks!. I implemented socket.io Authorization https://github.com/LearnBoost/socket.io/wiki/Authorizing in my project and wanted to share my solution as a way of saying thank you :)

elephant.io rocks!. I implemented socket.io Authorization https://github.com/LearnBoost/socket.io/wiki/Authorizing on my project and wanted to share my solution as a way of saying thank you :)
@guillaumepotier
Copy link
Member

Hi, thanks for your PR.

Looking at it, that seems related to #34 one, which is a bit more generalist because not auth-related and allowing more generally to add query params to url.

What do you think ?

Best

@Taluu
Copy link
Contributor

Taluu commented Jun 11, 2014

Closing this one, as #36 did the job and is more flexible (not surcharging the constructor with yet another argument...)

Thanks for the job though :)

@Taluu Taluu closed this Jun 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants