Make Kestrels Internals truly internal.

src/Servers/Kestrel/Core/src/Adapter/Internal/AdaptedPipeline.cs

@@ -5,14 +5,13 @@
 using System.IO;
 using System.IO.Pipelines;
 using System.Threading.Tasks;
-using Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http;
 using Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Infrastructure;
 using Microsoft.AspNetCore.Server.Kestrel.Transport.Abstractions.Internal;
 using Microsoft.Extensions.Logging;
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Adapter.Internal
 {
-    public class AdaptedPipeline : IDuplexPipe
+    internal class AdaptedPipeline : IDuplexPipe
     {
         private static readonly int MinAllocBufferSize = KestrelMemoryPool.MinimumSegmentSize / 2;
 

src/Servers/Kestrel/Core/src/Adapter/Internal/LoggingConnectionAdapter.cs

@@ -8,7 +8,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Adapter.Internal
 {
-    public class LoggingConnectionAdapter : IConnectionAdapter
+    internal class LoggingConnectionAdapter : IConnectionAdapter
     {
         private readonly ILogger _logger;
 

src/Servers/Kestrel/Core/src/Adapter/Internal/RawStream.cs

@@ -10,7 +10,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Adapter.Internal
 {
-    public class RawStream : Stream
+    internal class RawStream : Stream
     {
         private readonly PipeReader _input;
         private readonly PipeWriter _output;

src/Servers/Kestrel/Core/src/BadHttpRequestException.cs

@@ -1,4 +1,4 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System.Diagnostics;
@@ -42,10 +42,8 @@ internal static void Throw(RequestRejectionReason reason)
         }
 
         [StackTraceHidden]
-#pragma warning disable PUB0001 // Pubternal type in public API
-        public static void Throw(RequestRejectionReason reason, HttpMethod method)
+        internal static void Throw(RequestRejectionReason reason, HttpMethod method)
             => throw GetException(reason, method.ToString().ToUpperInvariant());
-#pragma warning restore PUB0001 // Pubternal type in public API
 
         [MethodImpl(MethodImplOptions.NoInlining)]
         internal static BadHttpRequestException GetException(RequestRejectionReason reason)
@@ -175,4 +173,4 @@ internal static BadHttpRequestException GetException(RequestRejectionReason reas
             return ex;
         }
     }
-}
+}

src/Servers/Kestrel/Core/src/Internal/CertificateLoader.cs → src/Servers/Kestrel/Core/src/CertificateLoader.cs

@@ -1,4 +1,4 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;

src/Servers/Kestrel/Core/src/Internal/ConnectionDispatcher.cs

@@ -1,4 +1,4 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
@@ -13,7 +13,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal
 {
-    public class ConnectionDispatcher : IConnectionDispatcher
+    internal class ConnectionDispatcher : IConnectionDispatcher
     {
         private static long _lastConnectionId = long.MinValue;
 

src/Servers/Kestrel/Core/src/Internal/ConnectionLimitMiddleware.cs

@@ -1,4 +1,4 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System.Threading.Tasks;
@@ -8,7 +8,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal
 {
-    public class ConnectionLimitMiddleware
+    internal class ConnectionLimitMiddleware
     {
         private readonly ConnectionDelegate _next;
         private readonly ResourceCounter _concurrentConnectionCounter;

src/Servers/Kestrel/Core/src/Internal/ConnectionLogScope.cs

@@ -8,7 +8,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal
 {
-    public class ConnectionLogScope : IReadOnlyList<KeyValuePair<string, object>>
+    internal class ConnectionLogScope : IReadOnlyList<KeyValuePair<string, object>>
     {
         private readonly string _connectionId;
 

src/Servers/Kestrel/Core/src/Internal/DuplexPipe.cs

@@ -1,4 +1,7 @@
-namespace System.IO.Pipelines
+// Copyright (c) Microsoft. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+namespace System.IO.Pipelines
 {
     internal class DuplexPipe : IDuplexPipe
     {

src/Servers/Kestrel/Core/src/Internal/Http/ConnectionOptions.cs

@@ -6,7 +6,7 @@
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
     [Flags]
-    public enum ConnectionOptions
+    internal enum ConnectionOptions
     {
         None = 0,
         Close = 1,

src/Servers/Kestrel/Core/src/Internal/Http/DateHeaderValueManager.cs

@@ -12,7 +12,7 @@ namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
     /// <summary>
     /// Manages the generation of the date header value.
     /// </summary>
-    public class DateHeaderValueManager : IHeartbeatHandler
+    internal class DateHeaderValueManager : IHeartbeatHandler
     {
         private static readonly byte[] _datePreambleBytes = Encoding.ASCII.GetBytes("\r\nDate: ");
 

src/Servers/Kestrel/Core/src/Internal/Http/Http1ChunkedEncodingMessageBody.cs

@@ -9,15 +9,14 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Connections;
-using Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Infrastructure;
 using Microsoft.AspNetCore.Server.Kestrel.Transport.Abstractions.Internal;
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
     /// <summary>
     ///   http://tools.ietf.org/html/rfc2616#section-3.6.1
     /// </summary>
-    public class Http1ChunkedEncodingMessageBody : Http1MessageBody
+    internal class Http1ChunkedEncodingMessageBody : Http1MessageBody
     {
         // byte consts don't have a data type annotation so we pre-cast it
         private const byte ByteCR = (byte)'\r';

src/Servers/Kestrel/Core/src/Internal/Http/Http1Connection.FeatureCollection.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public partial class Http1Connection : IHttpMinRequestBodyDataRateFeature,
+    internal partial class Http1Connection : IHttpMinRequestBodyDataRateFeature,
                                            IHttpMinResponseDataRateFeature
     {
         MinDataRate IHttpMinRequestBodyDataRateFeature.MinDataRate

src/Servers/Kestrel/Core/src/Internal/Http/Http1Connection.cs

@@ -11,11 +11,10 @@
 using Microsoft.AspNetCore.Connections;
 using Microsoft.AspNetCore.Http.Features;
 using Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Infrastructure;
-using Microsoft.AspNetCore.Server.Kestrel.Transport.Abstractions.Internal;
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public partial class Http1Connection : HttpProtocol, IRequestProcessor
+    internal partial class Http1Connection : HttpProtocol, IRequestProcessor
     {
         private const byte ByteAsterisk = (byte)'*';
         private const byte ByteForwardSlash = (byte)'/';

src/Servers/Kestrel/Core/src/Internal/Http/Http1ContentLengthMessageBody.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public class Http1ContentLengthMessageBody : Http1MessageBody
+    internal class Http1ContentLengthMessageBody : Http1MessageBody
     {
         private readonly long _contentLength;
         private long _inputLength;

src/Servers/Kestrel/Core/src/Internal/Http/Http1MessageBody.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public abstract class Http1MessageBody : MessageBody
+    internal abstract class Http1MessageBody : MessageBody
     {
         protected readonly Http1Connection _context;
 

src/Servers/Kestrel/Core/src/Internal/Http/Http1OutputProducer.cs

@@ -15,7 +15,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public class Http1OutputProducer : IHttpOutputProducer, IHttpOutputAborter, IDisposable
+    internal class Http1OutputProducer : IHttpOutputProducer, IHttpOutputAborter, IDisposable
     {
         // Use C#7.3's ReadOnlySpan<byte> optimization for static data https://vcsjones.com/2019/02/01/csharp-readonly-span-bytes-static/
         // "HTTP/1.1 100 Continue\r\n\r\n"

src/Servers/Kestrel/Core/src/Internal/Http/Http1ParsingHandler.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public readonly struct Http1ParsingHandler : IHttpRequestLineHandler, IHttpHeadersHandler
+    internal readonly struct Http1ParsingHandler : IHttpRequestLineHandler, IHttpHeadersHandler
     {
         public readonly Http1Connection Connection;
 

src/Servers/Kestrel/Core/src/Internal/Http/Http1UpgradeMessageBody.cs

@@ -12,7 +12,7 @@ namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
     /// The upgrade stream uses the raw connection stream instead of going through the RequestBodyPipe. This
     /// removes the redundant copy from the transport pipe to the body pipe.
     /// </summary>
-    public class Http1UpgradeMessageBody : Http1MessageBody
+    internal class Http1UpgradeMessageBody : Http1MessageBody
     {
         public bool _completed;
         public Http1UpgradeMessageBody(Http1Connection context)

src/Servers/Kestrel/Core/src/Internal/Http/HttpHeaders.Generated.cs

@@ -12,7 +12,7 @@
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
 
-    public partial class HttpRequestHeaders
+    internal partial class HttpRequestHeaders
     {
 
         private long _bits = 0;
@@ -4705,7 +4705,7 @@ public bool MoveNext()
         }
     }
 
-    public partial class HttpResponseHeaders
+    internal partial class HttpResponseHeaders
     {
         private static ReadOnlySpan<byte> HeaderBytes => new byte[]
         {
@@ -8531,7 +8531,7 @@ public bool MoveNext()
         }
     }
 
-    public partial class HttpResponseTrailers
+    internal partial class HttpResponseTrailers
     {
         private static ReadOnlySpan<byte> HeaderBytes => new byte[]
         {

src/Servers/Kestrel/Core/src/Internal/Http/HttpHeaders.cs

@@ -14,7 +14,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public abstract class HttpHeaders : IHeaderDictionary
+    internal abstract class HttpHeaders : IHeaderDictionary
     {
         protected long? _contentLength;
         protected bool _isReadOnly;

src/Servers/Kestrel/Core/src/Internal/Http/HttpMethod.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public enum HttpMethod: byte
+    internal enum HttpMethod: byte
     {
         Get,
         Put,

src/Servers/Kestrel/Core/src/Internal/Http/HttpParser.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public class HttpParser<TRequestHandler> : IHttpParser<TRequestHandler> where TRequestHandler : IHttpHeadersHandler, IHttpRequestLineHandler
+    internal class HttpParser<TRequestHandler> : IHttpParser<TRequestHandler> where TRequestHandler : IHttpHeadersHandler, IHttpRequestLineHandler
     {
         private bool _showErrorDetails;
 

src/Servers/Kestrel/Core/src/Internal/Http/HttpProtocol.FeatureCollection.cs

@@ -3,7 +3,6 @@
 
 using System;
 using System.Collections.Generic;
-using System.Diagnostics;
 using System.IO;
 using System.IO.Pipelines;
 using System.Net;
@@ -17,7 +16,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public partial class HttpProtocol : IHttpRequestFeature,
+    internal partial class HttpProtocol : IHttpRequestFeature,
                                         IHttpResponseFeature,
                                         IResponseBodyPipeFeature,
                                         IRequestBodyPipeFeature,

src/Servers/Kestrel/Core/src/Internal/Http/HttpProtocol.Generated.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public partial class HttpProtocol : IFeatureCollection
+    internal partial class HttpProtocol : IFeatureCollection
     {
         private static readonly Type IHttpRequestFeatureType = typeof(IHttpRequestFeature);
         private static readonly Type IHttpResponseFeatureType = typeof(IHttpResponseFeature);

src/Servers/Kestrel/Core/src/Internal/Http/HttpProtocol.cs

@@ -23,7 +23,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public abstract partial class HttpProtocol : IDefaultHttpContextContainer, IHttpResponseControl
+    internal abstract partial class HttpProtocol : IDefaultHttpContextContainer, IHttpResponseControl
     {
         private static readonly byte[] _bytesConnectionClose = Encoding.ASCII.GetBytes("\r\nConnection: close");
         private static readonly byte[] _bytesConnectionKeepAlive = Encoding.ASCII.GetBytes("\r\nConnection: keep-alive");

src/Servers/Kestrel/Core/src/Internal/Http/HttpRequestHeaders.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public sealed partial class HttpRequestHeaders : HttpHeaders
+    internal sealed partial class HttpRequestHeaders : HttpHeaders
     {
         private static long ParseContentLength(string value)
         {

src/Servers/Kestrel/Core/src/Internal/Http/HttpRequestPipeReader.cs

@@ -13,7 +13,7 @@ namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
     /// <summary>
     /// Default HttpRequest PipeReader implementation to be used by Kestrel.
     /// </summary>
-    public class HttpRequestPipeReader : PipeReader
+    internal class HttpRequestPipeReader : PipeReader
     {
         private MessageBody _body;
         private HttpStreamState _state;

src/Servers/Kestrel/Core/src/Internal/Http/HttpRequestTargetForm.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public enum HttpRequestTarget
+    internal enum HttpRequestTarget
     {
         Unknown = -1,
         // origin-form is the most common

src/Servers/Kestrel/Core/src/Internal/Http/HttpResponseHeaders.cs

@@ -12,7 +12,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public sealed partial class HttpResponseHeaders : HttpHeaders
+    internal sealed partial class HttpResponseHeaders : HttpHeaders
     {
         private static ReadOnlySpan<byte> _CrLf => new[] { (byte)'\r', (byte)'\n' };
         private static ReadOnlySpan<byte> _colonSpace => new[] { (byte)':', (byte)' ' };

src/Servers/Kestrel/Core/src/Internal/Http/HttpResponsePipeWriter.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public class HttpResponsePipeWriter : PipeWriter
+    internal class HttpResponsePipeWriter : PipeWriter
     {
         private HttpStreamState _state;
         private readonly IHttpResponseControl _pipeControl;

src/Servers/Kestrel/Core/src/Internal/Http/HttpResponseTrailers.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public partial class HttpResponseTrailers : HttpHeaders
+    internal partial class HttpResponseTrailers : HttpHeaders
     {
         public Enumerator GetEnumerator()
         {

src/Servers/Kestrel/Core/src/Internal/Http/HttpScheme.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public enum HttpScheme
+    internal enum HttpScheme
     {
         Unknown = -1,
         Http = 0,

src/Servers/Kestrel/Core/src/Internal/Http/HttpStreamState.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    enum HttpStreamState
+    internal enum HttpStreamState
     {
         Open,
         Closed,

src/Servers/Kestrel/Core/src/Internal/Http/HttpVersion.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public enum HttpVersion
+    internal enum HttpVersion
     {
         Unknown = -1,
         Http10 = 0,

src/Servers/Kestrel/Core/src/Internal/Http/IHttpHeadersHandler.cs

@@ -5,7 +5,7 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http
 {
-    public interface IHttpHeadersHandler
+    internal interface IHttpHeadersHandler
     {
         void OnHeader(Span<byte> name, Span<byte> value);
     }