Skip to content

Commit 51f9b66

Browse files
committed
Use int.MaxValue for gRPC message size limit (#424)
Use int.MaxValue for gRPC message size limit instead of grpcMaxMessageLength command line argument
1 parent 1f5d2bf commit 51f9b66

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/Messaging/MessagingStream.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,10 @@ internal class MessagingStream
1717
private readonly AsyncDuplexStreamingCall<StreamingMessage, StreamingMessage> _call;
1818
private readonly SemaphoreSlim _semaphoreSlim = new SemaphoreSlim(initialCount: 1, maxCount: 1);
1919

20-
internal MessagingStream(string host, int port, int maxMessageLength)
20+
internal MessagingStream(string host, int port)
2121
{
22+
const int maxMessageLength = int.MaxValue;
23+
2224
var channelOptions = new []
2325
{
2426
new ChannelOption(ChannelOptions.MaxReceiveMessageLength, maxMessageLength),

src/Worker.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public async static Task Main(string[] args)
3737
.WithParsed(ops => arguments = ops)
3838
.WithNotParsed(err => Environment.Exit(1));
3939

40-
var msgStream = new MessagingStream(arguments.Host, arguments.Port, arguments.MaxMessageLength);
40+
var msgStream = new MessagingStream(arguments.Host, arguments.Port);
4141
var requestProcessor = new RequestProcessor(msgStream);
4242

4343
// Send StartStream message
@@ -74,7 +74,7 @@ internal class WorkerArguments
7474
[Option("requestId", Required = true, HelpText = "Request ID used for gRPC communication with the Host.")]
7575
public string RequestId { get; set; }
7676

77-
[Option("grpcMaxMessageLength", Required = true, HelpText = "gRPC Maximum message size.")]
77+
[Option("grpcMaxMessageLength", Required = false, HelpText = "[Deprecated and ignored] gRPC Maximum message size.")]
7878
public int MaxMessageLength { get; set; }
7979
}
8080
}

0 commit comments

Comments
 (0)