nits

Author: jkotalik

src/Servers/IIS/IIS/src/Core/IISHttpContext.Log.cs

@@ -20,6 +20,9 @@ private static class Log
             private static readonly Action<ILogger, string, string, Exception> _unexpectedError =
                 LoggerMessage.Define<string, string>(LogLevel.Error, new EventId(3, "UnexpectedError"), @"Unexpected exception in ""{ClassName}.{MethodName}"".");
 
+            private static readonly Action<ILogger, string, string, Exception> _connectionBadRequest =
+                LoggerMessage.Define<string, string>(LogLevel.Information, new EventId(4, nameof(ConnectionBadRequest)), @"Connection id ""{ConnectionId}"" bad request data: ""{message}""");
+
             public static void ConnectionDisconnect(ILogger logger, string connectionId)
             {
                 _connectionDisconnect(logger, connectionId, null);
@@ -34,6 +37,11 @@ public static void UnexpectedError(ILogger logger, string className, Exception e
             {
                 _unexpectedError(logger, className, methodName, ex);
             }
+
+            public static void ConnectionBadRequest(ILogger logger, string connectionId, BadHttpRequestException ex)
+            {
+                _connectionBadRequest(logger, connectionId, ex.Message, ex);
+            }
         }
     }
 }

src/Servers/IIS/IIS/src/Core/IISHttpContext.cs

@@ -352,7 +352,7 @@ protected Task ProduceEnd()
         private void SetErrorResponseHeaders(int statusCode)
         {
             StatusCode = statusCode;
-            ReasonPhrase = string.Empty; // TODO
+            ReasonPhrase = string.Empty;
             HttpResponseHeaders.Clear();
         }
 
@@ -479,7 +479,7 @@ protected async Task FireOnCompleted()
 
         public void SetBadRequestState(BadHttpRequestException ex)
         {
-            //Log.ConnectionBadRequest(ConnectionId, ex);
+            Log.ConnectionBadRequest(_logger, RequestConnectionId, ex);
 
             if (!HasResponseStarted)
             {

src/Servers/IIS/IIS/src/IISServerOptions.cs

@@ -38,7 +38,6 @@ public class IISServerOptions
 
         internal string[] ServerAddresses { get; set; }
 
-
         // Matches the default maxAllowedContentLength in IIS (~28.6 MB)
         // https://www.iis.net/configreference/system.webserver/security/requestfiltering/requestlimits#005
         private long? _maxRequestBodySize = 30000000;

src/Servers/IIS/IIS/test/IISExpress.FunctionalTests/IISExpress.FunctionalTests.csproj

@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
   <Import Project="../FunctionalTest.props" />
 
@@ -11,7 +11,6 @@
 
 
   <ItemGroup>
-    <ProjectReference Include="..\..\src\Microsoft.AspNetCore.Server.IIS.csproj" />
     <ProjectReference Include="..\Common.Tests\Common.Tests.csproj" />
     <ProjectReference Include="$(RepositoryRoot)src\Hosting\Server.IntegrationTesting\src\Microsoft.AspNetCore.Server.IntegrationTesting.csproj" />
     <ProjectReference Include="$(RepositoryRoot)src\Servers\IIS\IntegrationTesting.IIS\src\Microsoft.AspNetCore.Server.IntegrationTesting.IIS.csproj" />