Firefox fix

bebo-dot-dev · bebo-dot-dev · commit 48ee5fc6fbdd · 2016-06-05T08:27:39.000+01:00
Overlay fix for Firefox and other clients that send additional data in the Connection header e.g. "Connection: Keep-Alive, Upgrade"
diff --git a/src/WebSocketsServer.cpp b/src/WebSocketsServer.cpp
@@ -590,7 +590,8 @@ void WebSocketsServer::handleHeader(WSclient_t * client, String * headerLine) {
             String headerValue = headerLine->substring(headerLine->indexOf(':') + 2);
 
             if(headerName.equalsIgnoreCase("Connection")) {
-                if(headerValue.equalsIgnoreCase("Upgrade")) {
+                headerValue.toLowerCase();
+            	if(headerValue.indexOf("upgrade") >= 0) {
                     client->cIsUpgrade = true;
                 }
             } else if(headerName.equalsIgnoreCase("Upgrade")) {

Original file line number	Diff line number	Diff line change
`@@ -590,7 +590,8 @@ void WebSocketsServer::handleHeader(WSclient_t * client, String * headerLine) {`
`590`	`590`	`String headerValue = headerLine->substring(headerLine->indexOf(':') + 2);`
`591`	`591`
`592`	`592`	`if(headerName.equalsIgnoreCase("Connection")) {`
`593`		`- if(headerValue.equalsIgnoreCase("Upgrade")) {`
	`593`	`+ headerValue.toLowerCase();`
	`594`	`+ if(headerValue.indexOf("upgrade") >= 0) {`
`594`	`595`	`client->cIsUpgrade = true;`
`595`	`596`	`}`
`596`	`597`	`} else if(headerName.equalsIgnoreCase("Upgrade")) {`