Status code encoding fix

Author: ladeak

src/CHttpServer/CHttpServer/Http3/QPackEncoder.cs

@@ -50,7 +50,7 @@ internal void Encode(int statusCode, Http3ResponseHeaderCollection headers, Pipe
         if (_statusCodesEncoderTable.TryGetValue(statusCode, out var knownHeader))
             EncodeIndexedFieldLine(knownHeader, destinationWriter);
         else
-            EncodeLiteralFieldWithLiteralValue(":status", statusCode.ToString(), destinationWriter);
+            EncodeIndexedFieldWithLiteralValue(_statusCodesEncoderTable.First().Value, statusCode.ToString(), destinationWriter);
 
         EncodeFieldLines(headers, destinationWriter);
     }

tests/CHttpServer.Tests/Http3/QPackDecoderTests.cs

@@ -274,32 +274,6 @@ public void Decode_Mix_DifferentHeaderOrder()
         Assert.Equal("longervalue", testHandler.Headers["longerheader"]);
     }
 
-    [Fact]
-    public async Task EncodeStatusCode200()
-    {
-        var stream = new MemoryStream();
-        var writer = new Http3FramingStreamWriter(stream, 1);
-        var sut = new QPackDecoder();
-        sut.Encode(200, new Http3ResponseHeaderCollection(), writer);
-        await writer.FlushAsync(TestContext.Current.CancellationToken);
-
-        byte[] expected = [0, 0, 25 | 0b1100_000];
-        expected.SequenceEqual(stream.ToArray());
-    }
-
-    [Fact]
-    public async Task EncodeStatusCode103()
-    {
-        var stream = new MemoryStream();
-        var writer = new Http3FramingStreamWriter(stream, 1);
-        var sut = new QPackDecoder();
-        sut.Encode(200, new Http3ResponseHeaderCollection(), writer);
-        await writer.FlushAsync(TestContext.Current.CancellationToken);
-
-        byte[] expected = [0, 0, 24 | 0b1100_000];
-        expected.SequenceEqual(stream.ToArray());
-    }
-
     private class TestQPackHeaderHandler : IQPackHeaderHandler
     {
         internal Dictionary<string, string> Headers { get; } = new();

tests/CHttpServer.Tests/Http3/QPackEncoderTests.cs

@@ -6,6 +6,31 @@ namespace CHttpServer.Tests.Http3;
 
 public class QPackEncoderTests
 {
+    [Fact]
+    public async Task EncodeStatusCode200()
+    {
+        var stream = new MemoryStream();
+        var writer = new Http3FramingStreamWriter(stream, 1);
+        var sut = new QPackDecoder();
+        sut.Encode(200, new Http3ResponseHeaderCollection(), writer);
+        await writer.FlushAsync(TestContext.Current.CancellationToken);
+
+        byte[] expected = [1, 3, 0, 0, 25 | 0b1100_0000];
+        Assert.True(expected.SequenceEqual(stream.ToArray()));
+    }
+
+    [Fact]
+    public async Task EncodeStatusCode103()
+    {
+        var stream = new MemoryStream();
+        var writer = new Http3FramingStreamWriter(stream, 2);
+        var sut = new QPackDecoder();
+        sut.Encode(103, new Http3ResponseHeaderCollection(), writer);
+        await writer.FlushAsync(TestContext.Current.CancellationToken);
+        byte[] expected = [2, 3, 0, 0, 24 | 0b1100_0000];
+        Assert.True(expected.SequenceEqual(stream.ToArray()));
+    }
+
     [Fact]
     public async Task EncodeLiteralFieldWithLiteralValue()
     {
@@ -62,7 +87,7 @@ public async Task EncodeIndexedFieldWithIndexedValue(byte index)
     }
 
     [Fact]
-    public async Task EncodeStatusCode()
+    public async Task EncodeStatusCodeRaw200()
     {
         var stream = new MemoryStream();
         var pipe = PipeWriter.Create(stream);
@@ -72,6 +97,17 @@ public async Task EncodeStatusCode()
         Assert.True(stream.ToArray().SequenceEqual(new byte[] { 0x00, 0x00, 0xD9 }));
     }
 
+    [Fact]
+    public async Task EncodeStatusCodeRaw8()
+    {
+        var stream = new MemoryStream();
+        var pipe = PipeWriter.Create(stream);
+        QPackDecoder sut = new();
+        sut.Encode(8, [], pipe);
+        await pipe.FlushAsync(TestContext.Current.CancellationToken);
+        Assert.True(stream.ToArray().SequenceEqual(new byte[] { 0x00, 0x00, 0x7F, 0x37, 0x01, 0x38 }));
+    }
+
     [Fact]
     public async Task EncodeWithoutStatusCode()
     {