Merge pull request #17 from jtattermusch/we-dont-need-no-backup-fix-unknown-handler

C# unknown method handler no longer needs to finish the request stream
pull/2090/head
Craig Tiller 10 years ago
commit 09bb58bbc7
  1. 1
      src/csharp/Grpc.Core/Internal/AsyncCallServer.cs
  2. 2
      src/csharp/Grpc.Core/Internal/ServerCallHandler.cs

@ -107,6 +107,7 @@ namespace Grpc.Core.Internal
call.StartSendStatusFromServer(status, HandleHalfclosed);
halfcloseRequested = true;
readingDone = true;
sendCompletionDelegate = completionDelegate;
}
}

@ -267,8 +267,6 @@ namespace Grpc.Core.Internal
var responseStream = new ServerResponseStream<byte[], byte[]>(asyncCall);
await responseStream.WriteStatusAsync(new Status(StatusCode.Unimplemented, "No such method."));
// TODO(jtattermusch): if we don't read what client has sent, the server call never gets disposed.
await requestStream.ToList();
await finishedTask;
}
}

Loading…
Cancel
Save