Fix credential NPE (#1955)

neozwu · web-flow · commit ada4cd50025c · 2017-04-20T17:18:23.000-07:00
* fix #1565
diff --git a/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java b/google-cloud-logging/src/main/java/com/google/cloud/logging/spi/v2/GrpcLoggingRpc.java
@@ -82,11 +82,12 @@ public GrpcLoggingRpc(LoggingOptions options) throws IOException {
       ChannelProvider channelProvider;
       // todo(mziccard): ChannelProvider should support null/absent credentials for testing
       if (options.getHost().contains("localhost")
-          || options.getCredentials().equals(NoCredentials.getInstance())) {
-        ManagedChannel managedChannel = ManagedChannelBuilder.forTarget(options.getHost())
-            .usePlaintext(true)
-            .executor(executor)
-            .build();
+          || NoCredentials.getInstance().equals(options.getCredentials())) {
+        ManagedChannel managedChannel =
+            ManagedChannelBuilder.forTarget(options.getHost())
+                .usePlaintext(true)
+                .executor(executor)
+                .build();
         channelProvider = FixedChannelProvider.create(managedChannel);
       } else {
         channelProvider = GrpcTransportOptions.setUpChannelProvider(
