Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Commit 99a4e5d

Browse files
committed
Fix #234
WIP Still needs tests
1 parent cd1c80d commit 99a4e5d

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

src/Microsoft.AspNet.Server.Kestrel/Infrastructure/MemoryPoolIterator2.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -442,6 +442,7 @@ public MemoryPoolIterator2 CopyTo(byte[] array, int offset, int count, out int a
442442
else
443443
{
444444
Buffer.BlockCopy(block.Array, index, array, offset, following);
445+
offset += following;
445446
remaining -= following;
446447
block = block.Next;
447448
index = block.Start;

test/Microsoft.AspNet.Server.Kestrel.FunctionalTests/RequestTests.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@
88
using Microsoft.AspNet.Builder;
99
using Microsoft.AspNet.Hosting;
1010
using Microsoft.AspNet.Http;
11+
using Microsoft.AspNet.Http.Features;
12+
using Microsoft.AspNet.Server.Kestrel.Filter;
1113
using Microsoft.Dnx.Runtime.Infrastructure;
1214
using Microsoft.Extensions.Configuration;
1315
using Xunit;
@@ -16,7 +18,7 @@ namespace Microsoft.AspNet.Server.Kestrel.FunctionalTests
1618
{
1719
public class RequestTests
1820
{
19-
[Fact(Skip = "https://github.com/aspnet/KestrelHttpServer/issues/234")]
21+
[Fact]
2022
public async Task LargeUpload()
2123
{
2224
var config = new ConfigurationBuilder()
@@ -30,6 +32,9 @@ public async Task LargeUpload()
3032
hostBuilder.UseServer("Microsoft.AspNet.Server.Kestrel");
3133
hostBuilder.UseStartup(app =>
3234
{
35+
var info = app.ServerFeatures.Get<IKestrelServerInformation>();
36+
info.ConnectionFilter = new NoOpConnectionFilter();
37+
3338
app.Run(async context =>
3439
{
3540
// Read the full request body

0 commit comments

Comments
 (0)