grpc-js: do not pass target name override to proxy

grpc · Apr 18, 2020 · a3a7534 · a3a7534
1 parent ddb968e
commit a3a7534
Showing 1 changed file with 6 additions and 15 deletions.
diff --git a/packages/grpc-js/src/subchannel.ts b/packages/grpc-js/src/subchannel.ts
@@ -416,21 +416,12 @@ export class Subchannel {
       this.credentials._getConnectionOptions() || {};
 
     if ('secureContext' in connectionOptions) {
-      // If provided, the value of grpc.ssl_target_name_override should be used
-      // to override the target hostname when checking server identity.
-      // This option is used for testing only.
-      if (this.options['grpc.ssl_target_name_override']) {
-        const sslTargetNameOverride = this.options[
-          'grpc.ssl_target_name_override'
-        ]!;
-        connectionOptions.checkServerIdentity = (
-          host: string,
-          cert: PeerCertificate
-        ): Error | undefined => {
-          return checkServerIdentity(sslTargetNameOverride, cert);
-        };
-        connectionOptions.servername = sslTargetNameOverride;
-      }
+      connectionOptions.checkServerIdentity = (
+        host: string,
+        cert: PeerCertificate
+      ): Error | undefined => {
+        return checkServerIdentity(host, cert);
+      };
     }
 
     getProxiedConnection(this.subchannelAddress, this.options, connectionOptions).then(