-
Notifications
You must be signed in to change notification settings - Fork 603
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to enable server certificate verification on the Nodejs clients like MetricServiceClient/KeyManagementServiceClient for Mutual TLS ? #5453
Comments
I am also facing the similar issue. Did you find any solution? |
I don't think this can be done through grpc, see. However, I do think this could be done using our REST transport. You should be able to configure a client to use mTLS or use the agent parameter like bigquery. In order to do this you'll need to instantiate using the googleapis library, like so:
and in the request, you can send a second parameter (options) that extends GaxiosOptions |
@Ruthwik did this work for you? |
@sofisl I will post here once I try these changes. It requires a change of our current libraries.
Is it also the same for monitoring client? |
Yes, it should. Closing for now to try and keep queue clean - please open a new issue if this doesn't work for you! |
We are using Nodejs MetricServiceClient/KeyManagementServiceClient client in our product. How do we enable server certificate verification in the MetricServiceClient/KeyManagementServiceClient client?
We want to enable client-side verification of server certificates with our CA bundle.
We've explored the options of MetricServiceClient/KeyManagementServiceClient but couldn't find any.
In the case of other cloud providers we have found a way to pass the CA bundle in the options where the client verifies.
Example for AWS
In the above example, If rejectUnauthorized is true the server will reject any connection which is not authorized with the list of supplied CAs. I also found the it is different for services and client in google nodejs SDK.
@beccasaurus @blowmage
The text was updated successfully, but these errors were encountered: