-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added extra headers #35
base: master
Are you sure you want to change the base?
Conversation
Allowed the creation of WebSocket with personal headers like HTTP Bearer Authentication
Sorry I don't know that I want this... The correct way to do webSocket authentication is to have a authentication packet of your own format you send first before doing anything else. Even though it makes since to send headers with webSocket, its not really supported anywhere. Chrome/Firefox don't support it. Although its slowly making it into the spec I don't feel its there yet: https://bugzilla.mozilla.org/show_bug.cgi?id=1229443 I feel like providing a way for people to send headers and auth is basically a trap, as it is so badly supported anywhere. I don't want to lead people into a trap. |
Hi,
there are no reasons to limit custom headers IMHO. |
Any news? |
I also want to customize the headers I send for authenticating my custom ws client to my custom ws server. |
Hello,
First of all, thank you for this library. It's really useful.
The reason of this pull request is that in my scenario I need to authenticate a websocket client with HTTP Basic Authentication, but at the moment it is not possible to add a personal header in an easy way when the websocket is created.
So I propose to add a list of personal headers when the websocket is created.
I've already tested this new feature in my project and it works fine.