Remove IKestrelTrace (#37242)

Author: JamesNK

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

@@ -24,7 +24,7 @@ public ConnectionDispatcher(ServiceContext serviceContext, Func<T, Task> connect
             _transportConnectionManager = transportConnectionManager;
         }
 
-        private IKestrelTrace Log => _serviceContext.Log;
+        private KestrelTrace Log => _serviceContext.Log;
 
         public Task StartAcceptingConnections(IConnectionListener<T> listener)
         {

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

@@ -34,7 +34,7 @@ internal class Http1OutputProducer : IHttpOutputProducer, IDisposable
         private readonly string _connectionId;
         private readonly BaseConnectionContext _connectionContext;
         private readonly MemoryPool<byte> _memoryPool;
-        private readonly IKestrelTrace _log;
+        private readonly KestrelTrace _log;
         private readonly IHttpMinResponseDataRateFeature _minResponseDataRateFeature;
         private readonly IHttpOutputAborter _outputAborter;
         private readonly TimingPipeFlusher _flusher;
@@ -78,7 +78,7 @@ public Http1OutputProducer(
             string connectionId,
             BaseConnectionContext connectionContext,
             MemoryPool<byte> memoryPool,
-            IKestrelTrace log,
+            KestrelTrace log,
             ITimeoutControl timeoutControl,
             IHttpMinResponseDataRateFeature minResponseDataRateFeature,
             IHttpOutputAborter outputAborter)

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

@@ -101,7 +101,7 @@ public void Initialize(HttpConnectionContext context)
         public IFeatureCollection ConnectionFeatures => _context.ConnectionFeatures;
         public IHttpOutputProducer Output { get; protected set; } = default!;
 
-        protected IKestrelTrace Log => ServiceContext.Log;
+        protected KestrelTrace Log => ServiceContext.Log;
         private DateHeaderValueManager DateHeaderValueManager => ServiceContext.DateHeaderValueManager;
         // Hold direct reference to ServerOptions since this is used very often in the request processing path
         protected KestrelServerOptions ServerOptions { get; set; } = default!;

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

@@ -42,7 +42,7 @@ protected MessageBody(HttpProtocol context)
 
         public virtual bool IsEmpty => false;
 
-        protected IKestrelTrace Log => _context.ServiceContext.Log;
+        protected KestrelTrace Log => _context.ServiceContext.Log;
 
         public abstract ValueTask<ReadResult> ReadAsync(CancellationToken cancellationToken = default);
 

src/Servers/Kestrel/Core/src/Internal/Http2/Http2Connection.cs

@@ -139,7 +139,7 @@ public Http2Connection(HttpConnectionContext context)
 
         public PipeReader Input => _input.Reader;
 
-        public IKestrelTrace Log => _context.ServiceContext.Log;
+        public KestrelTrace Log => _context.ServiceContext.Log;
         public IFeatureCollection ConnectionFeatures => _context.ConnectionFeatures;
         public ISystemClock SystemClock => _context.ServiceContext.SystemClock;
         public ITimeoutControl TimeoutControl => _context.TimeoutControl;

src/Servers/Kestrel/Core/src/Internal/Http2/Http2FrameWriter.cs

@@ -34,7 +34,7 @@ internal class Http2FrameWriter
         private readonly Http2Connection _http2Connection;
         private readonly OutputFlowControl _connectionOutputFlowControl;
         private readonly string _connectionId;
-        private readonly IKestrelTrace _log;
+        private readonly KestrelTrace _log;
         private readonly ITimeoutControl _timeoutControl;
         private readonly MinDataRate? _minResponseDataRate;
         private readonly TimingPipeFlusher _flusher;

src/Servers/Kestrel/Core/src/Internal/Http2/Http2OutputProducer.cs

@@ -23,7 +23,7 @@ internal class Http2OutputProducer : IHttpOutputProducer, IHttpOutputAborter, IV
         private int StreamId => _stream.StreamId;
         private readonly Http2FrameWriter _frameWriter;
         private readonly TimingPipeFlusher _flusher;
-        private readonly IKestrelTrace _log;
+        private readonly KestrelTrace _log;
 
         // This should only be accessed via the FrameWriter. The connection-level output flow control is protected by the
         // FrameWriter's connection-level write lock.

src/Servers/Kestrel/Core/src/Internal/Http3/Http3Connection.cs

@@ -83,7 +83,7 @@ private void UpdateHighestOpenedRequestStreamId(long streamId)
         // https://quicwg.org/base-drafts/draft-ietf-quic-http.html#section-5.2-2
         private long GetCurrentGoAwayStreamId() => Interlocked.Read(ref _highestOpenedRequestStreamId) + 4;
 
-        private IKestrelTrace Log => _context.ServiceContext.Log;
+        private KestrelTrace Log => _context.ServiceContext.Log;
         public KestrelServerLimits Limits => _context.ServiceContext.ServerOptions.Limits;
         public Http3ControlStream? OutboundControlStream { get; set; }
         public Http3ControlStream? ControlStream { get; set; }

src/Servers/Kestrel/Core/src/Internal/Http3/Http3ControlStream.cs

@@ -65,7 +65,7 @@ private void OnStreamClosed()
         }
 
         public PipeReader Input => _context.Transport.Input;
-        public IKestrelTrace Log => _context.ServiceContext.Log;
+        public KestrelTrace Log => _context.ServiceContext.Log;
 
         public long StreamTimeoutTicks { get; set; }
         public bool IsReceivingHeader => _headerType == -1;

src/Servers/Kestrel/Core/src/Internal/Http3/Http3FrameWriter.cs

@@ -41,7 +41,7 @@ internal class Http3FrameWriter
         private readonly ITimeoutControl _timeoutControl;
         private readonly MinDataRate? _minResponseDataRate;
         private readonly MemoryPool<byte> _memoryPool;
-        private readonly IKestrelTrace _log;
+        private readonly KestrelTrace _log;
         private readonly IStreamIdFeature _streamIdFeature;
         private readonly IHttp3Stream _http3Stream;
         private readonly Http3RawFrame _outgoingFrame;
@@ -61,7 +61,7 @@ internal class Http3FrameWriter
         private bool _completed;
         private bool _aborted;
 
-        public Http3FrameWriter(ConnectionContext connectionContext, ITimeoutControl timeoutControl, MinDataRate? minResponseDataRate, MemoryPool<byte> memoryPool, IKestrelTrace log, IStreamIdFeature streamIdFeature, Http3PeerSettings clientPeerSettings, IHttp3Stream http3Stream)
+        public Http3FrameWriter(ConnectionContext connectionContext, ITimeoutControl timeoutControl, MinDataRate? minResponseDataRate, MemoryPool<byte> memoryPool, KestrelTrace log, IStreamIdFeature streamIdFeature, Http3PeerSettings clientPeerSettings, IHttp3Stream http3Stream)
         {
             _connectionContext = connectionContext;
             _timeoutControl = timeoutControl;