feat: add google-c2p dependence to DirectPath

googleapis · Oct 15, 2021 · ce1227a · ce1227a
1 parent 119b6d1
commit ce1227a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/gax-grpc/src/main/java/com/google/api/gax/grpc/InstantiatingGrpcChannelProvider.java b/gax-grpc/src/main/java/com/google/api/gax/grpc/InstantiatingGrpcChannelProvider.java
@@ -331,12 +331,12 @@ private ManagedChannel createSingleChannel() throws IOException, GeneralSecurity
 
     ManagedChannelBuilder<?> builder;
 
-    // TODO(weiranf): Add API in ComputeEngineCredentials to check default service account.
-    String directPathXdsEnv = envProvider.getenv(DIRECT_PATH_ENABLE_XDS);
-    boolean isDirectPathXdsEnabled = Boolean.parseBoolean(directPathXdsEnv);
+    // Check DirectPath traffic.
+    boolean isDirectPathXdsEnabled = false;
     if (isDirectPathEnabled(serviceAddress)
         && isNonDefaultServiceAccountAllowed()
         && isOnComputeEngine()) {
+      isDirectPathXdsEnabled = Boolean.parseBoolean(envProvider.getenv(DIRECT_PATH_ENABLE_XDS));
       if (isDirectPathXdsEnabled) {
         // google-c2p resolver target must not have a port number
         builder = ComputeEngineChannelBuilder.forTarget("google-c2p:///" + serviceAddress);