fix a possible-null warning

accidentally introduced in PR #917, somehow I overlooked the warning
connamara · Dec 31, 2024 · 68b6fb5 · 68b6fb5
1 parent 124e0fe
commit 68b6fb5
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/QuickFIXn/Transport/StreamFactory.cs b/QuickFIXn/Transport/StreamFactory.cs
@@ -72,6 +72,10 @@ internal static Stream CreateClientStream(IPEndPoint endpoint, SocketSettings se
 
             if (!settings.SocketIgnoreProxy)
             {
+                if (settings.ServerCommonName is null) {
+                    throw new ConfigError(
+                        $"{SessionSettings.SOCKET_IGNORE_PROXY}=Y needs {SessionSettings.SSL_SERVERNAME} to be set");
+                }
                 // If system has configured a proxy for this config, use it.
                 socket = CreateTunnelThruProxy(endpoint.Address.ToString(), endpoint.Port, settings.ServerCommonName);
             }