fix #32 by handling connection header correctly (#33)

This commit is contained in:
Andrew Stanton-Nurse 2016-11-15 16:22:50 -08:00 committed by GitHub
parent 2431c5925c
commit f3ebe03a0b
1 changed files with 10 additions and 12 deletions

View File

@ -21,11 +21,7 @@ namespace Microsoft.AspNetCore.WebSockets.Internal
return false; return false;
} }
foreach (var pair in request.Headers) foreach (var value in request.Headers.GetCommaSeparatedValues(Constants.Headers.Connection))
{
if (string.Equals(Constants.Headers.Connection, pair.Key, StringComparison.OrdinalIgnoreCase))
{
foreach (var value in pair.Value)
{ {
if (string.Equals(Constants.Headers.ConnectionUpgrade, value, StringComparison.OrdinalIgnoreCase)) if (string.Equals(Constants.Headers.ConnectionUpgrade, value, StringComparison.OrdinalIgnoreCase))
{ {
@ -33,8 +29,10 @@ namespace Microsoft.AspNetCore.WebSockets.Internal
break; break;
} }
} }
}
else if (string.Equals(Constants.Headers.Upgrade, pair.Key, StringComparison.OrdinalIgnoreCase)) foreach (var pair in request.Headers)
{
if (string.Equals(Constants.Headers.Upgrade, pair.Key, StringComparison.OrdinalIgnoreCase))
{ {
if (string.Equals(Constants.Headers.UpgradeWebSocket, pair.Value, StringComparison.OrdinalIgnoreCase)) if (string.Equals(Constants.Headers.UpgradeWebSocket, pair.Value, StringComparison.OrdinalIgnoreCase))
{ {