HTTP/3: Fix variable length encoding (#51486)

dotnet · Apr 19, 2021 · 1f89aba · 1f89aba
1 parent 62d3bac
commit 1f89aba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/...raries/Common/src/System/Net/Http/aspnetcore/Http3/Helpers/VariableLengthIntegerHelper.cs b/...raries/Common/src/System/Net/Http/aspnetcore/Http3/Helpers/VariableLengthIntegerHelper.cs
@@ -33,7 +33,7 @@ internal static class VariableLengthIntegerHelper
 
         // public for internal use in aspnetcore
         public const uint OneByteLimit = (1U << 6) - 1;
-        public const uint TwoByteLimit = (1U << 16) - 1;
+        public const uint TwoByteLimit = (1U << 14) - 1;
         public const uint FourByteLimit = (1U << 30) - 1;
         public const long EightByteLimit = (1L << 62) - 1;
 

diff --git a/src/libraries/Common/src/System/Net/Http/aspnetcore/Http3/QPack/HeaderField.cs b/src/libraries/Common/src/System/Net/Http/aspnetcore/Http3/QPack/HeaderField.cs
@@ -5,6 +5,10 @@ namespace System.Net.Http.QPack
 {
     internal readonly struct HeaderField
     {
+        // https://quicwg.org/base-drafts/draft-ietf-quic-http.html#section-4.1.1.3-1
+        // public for internal use in aspnetcore
+        public const int RfcOverhead = 32;
+
         public HeaderField(byte[] name, byte[] value)
         {
             Name = name;