From ab3d3bd7b50d75cf1bd48a3106a460a63d9b2789 Mon Sep 17 00:00:00 2001 From: David Burhans Date: Tue, 9 Jun 2015 08:06:04 -0600 Subject: [PATCH] fix spacing. add intermediate variables --- websocket-sharp/HttpRequest.cs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/websocket-sharp/HttpRequest.cs b/websocket-sharp/HttpRequest.cs index d7c2c842..d6a11d67 100644 --- a/websocket-sharp/HttpRequest.cs +++ b/websocket-sharp/HttpRequest.cs @@ -133,8 +133,12 @@ namespace WebSocketSharp var headers = req.Headers; headers["Upgrade"] = "websocket"; headers["Connection"] = "Upgrade"; - bool isDefaultPort = (uri.Port == 80 && uri.Scheme == "ws") || (uri.Port == 443 && uri.Scheme == "wss"); - headers["Host"] = isDefaultPort ? uri.DnsSafeHost : uri.Authority; + var port = uri.Port; + var scheme = uri.Scheme + bool isDefaultPort = (port == 80 && scheme == "ws") || (port == 443 && scheme == "wss"); + // only include port in host header if it is non-default + // https://tools.ietf.org/html/rfc6455#page-17 + headers["Host"] = isDefaultPort ? uri.DnsSafeHost : uri.Authority; return req; }