From 2246c6b55b371296adb6c8edcb6ada8885c410d0 Mon Sep 17 00:00:00 2001 From: Alfred Myers Date: Tue, 28 Aug 2018 23:01:51 -0300 Subject: [PATCH] Fixed comment and log message for ForwardedHeadersMiddleware (#357) - Updated comment to reflect the addition of RequireHeaderSymmetry - Fixed header name on log message --- .../ForwardedHeadersMiddleware.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Microsoft.AspNetCore.HttpOverrides/ForwardedHeadersMiddleware.cs b/src/Microsoft.AspNetCore.HttpOverrides/ForwardedHeadersMiddleware.cs index decda81d58..6302917bcf 100644 --- a/src/Microsoft.AspNetCore.HttpOverrides/ForwardedHeadersMiddleware.cs +++ b/src/Microsoft.AspNetCore.HttpOverrides/ForwardedHeadersMiddleware.cs @@ -146,7 +146,7 @@ namespace Microsoft.AspNetCore.HttpOverrides public void ApplyForwarders(HttpContext context) { - // Gather expected headers. Enabled headers must have the same number of entries. + // Gather expected headers. string[] forwardedFor = null, forwardedProto = null, forwardedHost = null; bool checkFor = false, checkProto = false, checkHost = false; int entryCount = 0; @@ -282,7 +282,7 @@ namespace Microsoft.AspNetCore.HttpOverrides } else if (_options.RequireHeaderSymmetry) { - _logger.LogWarning(4, $"Incorrect number of x-forwarded-proto header values, see {nameof(_options.RequireHeaderSymmetry)}."); + _logger.LogWarning(4, $"Incorrect number of x-forwarded-host header values, see {nameof(_options.RequireHeaderSymmetry)}."); return; } }