Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ws communication headers are compared case sensitive #22

Open
amandel opened this issue May 22, 2020 · 0 comments
Open

ws communication headers are compared case sensitive #22

amandel opened this issue May 22, 2020 · 0 comments

Comments

@amandel
Copy link

amandel commented May 22, 2020

Http header names are not case sensitive, bool MQTTWSTraits::connect(WiFiClient* clientIn, const char* host, int port, const char *path) does a case sensitive coparison of the headers (temp.startsWith("upgrade: websocket").
If a client sends the header in different casing as expected, the websocket connection will fail.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@amandel