always set default max message size for now

momentohq · Feb 28, 2024 · 9120504 · 9120504
1 parent 46bc475
commit 9120504
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Momento.Sdk/Internal/Utils.cs b/src/Momento.Sdk/Internal/Utils.cs
@@ -36,8 +36,10 @@ public static GrpcChannelOptions GrpcChannelOptionsFromGrpcConfig(IGrpcConfigura
         var channelOptions = grpcConfig?.GrpcChannelOptions ?? new GrpcChannelOptions();
         channelOptions.Credentials = ChannelCredentials.SecureSsl;
         channelOptions.LoggerFactory ??= loggerFactory;
-        channelOptions.MaxReceiveMessageSize ??= DEFAULT_MAX_MESSAGE_SIZE;
-        channelOptions.MaxSendMessageSize ??= DEFAULT_MAX_MESSAGE_SIZE;
+        // TODO: how to know if user set the value or if max receive size is just set to the default 4mb or null, respectively?
+        // Currently just overwriting to 5mb
+        channelOptions.MaxReceiveMessageSize = DEFAULT_MAX_MESSAGE_SIZE;
+        channelOptions.MaxSendMessageSize = DEFAULT_MAX_MESSAGE_SIZE;
 #if NET5_0_OR_GREATER
         channelOptions.HttpHandler = new System.Net.Http.SocketsHttpHandler
         {