Skip to content

Commit cf3ca5f

Browse files
committed
fix #6
Firefox send more then Upgrade in Connection header
1 parent 28b7619 commit cf3ca5f

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

src/WebSocketsServer.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -402,8 +402,11 @@ void WebSocketsServer::handleHeader(WSclient_t * client) {
402402
if(headerLine.startsWith("GET ")) {
403403
// cut URL out
404404
client->cUrl = headerLine.substring(4, headerLine.indexOf(' ', 4));
405-
} else if(headerLine == "Connection: Upgrade") {
406-
client->cIsUpgrade = true;
405+
} else if(headerLine.startsWith("Connection: ")) {
406+
// 12 = lenght of "Connection: "
407+
if(headerLine.indexOf("Upgrade", 12)) {
408+
client->cIsUpgrade = true;
409+
}
407410
} else if(headerLine.startsWith("Upgrade: ")) {
408411
// 9 = lenght of "Upgrade: "
409412
String low = headerLine.substring(9);

tests/webSocket.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<head>
33

44
<script>
5-
var connection = new WebSocket('ws://10.11.2.1:81/test', ['arduino']);
5+
var connection = new WebSocket('ws://10.11.2.2:81/test', ['arduino']);
66

77
connection.onopen = function () {
88
connection.send('Message from Browser to ESP8266 yay its Working!! ' + new Date());

0 commit comments

Comments
 (0)