Merged

fix: never use credentials in combination with plain text #98

Changes from all commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Failed to load files.
Original file line numberDiff line numberDiff line change
Expand Up@@ -20,6 +20,7 @@
import com.google.api.gax.grpc.GrpcInterceptorProvider;
import com.google.api.gax.retrying.RetrySettings;
import com.google.api.gax.rpc.TransportChannelProvider;
import com.google.cloud.NoCredentials;
import com.google.cloud.ServiceDefaults;
import com.google.cloud.ServiceOptions;
import com.google.cloud.ServiceRpc;
Expand DownExpand Up@@ -422,6 +423,8 @@ public ManagedChannelBuilder apply(ManagedChannelBuilder builder) {
return builder.usePlaintext();
}
});
// As we are using plain text, we should never send any credentials.
this.setCredentials(NoCredentials.getInstance());
}
return new SpannerOptions(this);
}
Expand Down