Skip to content

Commit 4fbf018

Browse files
committed
Rename enable setting
1 parent 3dbecd3 commit 4fbf018

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

src/Servers/Kestrel/Core/src/Http2Limits.cs

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,5 +141,13 @@ public int InitialStreamWindowSize
141141
_initialStreamWindowSize = value;
142142
}
143143
}
144+
145+
/// <summary>
146+
/// Gets or sets a value that controls whether dynamic compression of response headers is enabled.
147+
/// </summary>
148+
/// <remarks>
149+
/// Defaults to true.
150+
/// </remarks>
151+
public bool EnableResponseHeaderCompression { get; set; } = true;
144152
}
145153
}

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public Http2FrameWriter(
7171
_outgoingFrame = new Http2Frame();
7272
_headerEncodingBuffer = new byte[_maxFrameSize];
7373

74-
_hpackEncoder = new HPackEncoder(serviceContext.ServerOptions.DisableResponseDynamicHeaderCompression);
74+
_hpackEncoder = new HPackEncoder(!serviceContext.ServerOptions.Limits.Http2.EnableResponseHeaderCompression);
7575
}
7676

7777
public void UpdateMaxHeaderTableSize(uint maxHeaderTableSize)

src/Servers/Kestrel/Core/src/KestrelServerOptions.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,6 @@ public class KestrelServerOptions
5555
/// </remarks>
5656
public bool DisableStringReuse { get; set; } = false;
5757

58-
/// <summary>
59-
/// Gets or sets a value that controls whether dynamic compression of response headers is enabled across HTTP/2 and HTTP/3 requests.
60-
/// </summary>
61-
/// <remarks>
62-
/// Defaults to false.
63-
/// </remarks>
64-
public bool DisableResponseDynamicHeaderCompression { get; set; } = false;
65-
6658
/// <summary>
6759
/// Enables the Listen options callback to resolve and use services registered by the application during startup.
6860
/// Typically initialized by UseKestrel()"/>.

src/Servers/Kestrel/test/InMemory.FunctionalTests/Http2/Http2ConnectionTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1909,7 +1909,7 @@ await ExpectAsync(Http2FrameType.HEADERS,
19091909
[Fact]
19101910
public async Task HEADERS_DisableDynamicHeaderCompression_HeadersNotCompressed()
19111911
{
1912-
_serviceContext.ServerOptions.DisableResponseDynamicHeaderCompression = true;
1912+
_serviceContext.ServerOptions.Limits.Http2.EnableResponseHeaderCompression = false;
19131913

19141914
await InitializeConnectionAsync(_noopApplication);
19151915

0 commit comments

Comments
 (0)