diff --git a/src/HotChocolate/Adapters/src/Adapters.OpenApi.Core/Execution/DynamicEndpointMiddleware.cs b/src/HotChocolate/Adapters/src/Adapters.OpenApi.Core/Execution/DynamicEndpointMiddleware.cs index 00e391ca515..c4a5d7734cf 100644 --- a/src/HotChocolate/Adapters/src/Adapters.OpenApi.Core/Execution/DynamicEndpointMiddleware.cs +++ b/src/HotChocolate/Adapters/src/Adapters.OpenApi.Core/Execution/DynamicEndpointMiddleware.cs @@ -159,14 +159,9 @@ await Results.Problem( var memory = writer.GetMemory(chunkSize); read = await body.ReadAsync(memory, cancellationToken).ConfigureAwait(false); writer.Advance(read); + } while (read != 0); - // if (_maxRequestSize < writer.Length) - // { - // throw DefaultHttpRequestParser_MaxRequestSizeExceeded(); - // } - } while (read == chunkSize); - - if (read == 0) + if (writer.Length == 0) { throw new BadRequestException("Expected to have a body"); } diff --git a/src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Parsers/DefaultHttpRequestParser.cs b/src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Parsers/DefaultHttpRequestParser.cs index 9576c41c2ae..99cf7d30fc5 100644 --- a/src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Parsers/DefaultHttpRequestParser.cs +++ b/src/HotChocolate/AspNetCore/src/AspNetCore.Pipeline/Parsers/DefaultHttpRequestParser.cs @@ -60,7 +60,7 @@ public async ValueTask ParsePersistedOperationRequestAsync( { const int chunkSize = 256; using var writer = new PooledArrayWriter(); - var read = 0; + int read; do { @@ -72,7 +72,7 @@ public async ValueTask ParsePersistedOperationRequestAsync( { throw DefaultHttpRequestParser_MaxRequestSizeExceeded(); } - } while (read == chunkSize); + } while (read != 0); if (writer.Length == 0) { @@ -309,7 +309,7 @@ private async ValueTask> ReadAsync( { throw DefaultHttpRequestParser_MaxRequestSizeExceeded(); } - } while (read == chunkSize); + } while (read != 0); if (writer.Length == 0) {