-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
Private repo with SSL certificate and key #35
Comments
You said it all. I'd also like this to work with these certificate options. |
As a temporary fix, we could pass the https://nodejs.org/api/tls.html#tls_new_tls_tlssocket_socket_options |
Create a new module or add the logic to this module. Up to you.
I don't think it would fit into the goal of that module.
Got passes all option down to https://nodejs.org/api/https.html#https_https_request_options_callback |
While debugging sindresorhus/update-notifier#100 I noticed this module does not use npm-config ssl properties:
I want to create a PR for this issue how would you proceed?
registry-url
orregistry-auth-token
which returns this informationgot
. Is it possible to switch the request lib?The text was updated successfully, but these errors were encountered: