Skip to content
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

SSL options #22

Closed
crazed opened this issue Jun 8, 2016 · 5 comments
Closed

SSL options #22

crazed opened this issue Jun 8, 2016 · 5 comments

Comments

@crazed
Copy link

crazed commented Jun 8, 2016

Currently if you use a self-signed certificate, I don't see an easy way to use this with node-vault. The only way seems to be:

var https = require('https');
https.globalAgent.options = {
    rejectUnauthorized: false,
    ca: <string of ca>,
};

The Vault.prototype._handleCallback function seems to mangle the original options passed into individual requests before passing on to the request module.

@BeneStem
Copy link

Got this error now too.
Would be great if we could get a fix soonish :P
If you want me to help I can make a PR to fix it...
Greetz
/ben

@kr1sp1n
Copy link
Collaborator

kr1sp1n commented Jul 6, 2016

@crazed @BeneStem What about this example here: https://github.com/kr1sp1n/node-vault/blob/c81353ead66ffb5403dfaca70d125917f3c05695/example/pass_request_options.js

You can pass agentOptions to any request but would this solve the issue with the self-signed certs?

@BeneStem
Copy link

BeneStem commented Jul 7, 2016

I haven't tried the new version yet.

With the old coffeescript non-promise version the request options where overwritten.
There is a bug that the options are set into options.json and then option.json is set to null.

I will try the new version in 2 weeks when I am back from vacation.

Is it released yet?

Thanks for your reply and the example :)

Greetz
/ben

On 7 Jul 2016, at 00:02, Krispin Schulz notifications@github.com wrote:

@crazed @BeneStem What about this example here: https://github.com/kr1sp1n/node-vault/blob/c81353ead66ffb5403dfaca70d125917f3c05695/example/pass_request_options.js

You can pass agentOptions to any request but would this solve the issue with the self-signed certs?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@kr1sp1n
Copy link
Collaborator

kr1sp1n commented Jul 12, 2016

@BeneStem Yes, it is released. Latest version is 0.5.1
Would love to hear if this fixed your issue.

@BeneStem
Copy link

Thanks! I think you can close the issue for now. When I try it on Saturday I might open a new one. But it looks like it should work fine ;)

On 12 Jul 2016, at 09:21, Krispin Schulz notifications@github.com wrote:

@BeneStem Yes, it is released. Latest version is 0.5.1
Would love to hear if this fixed your issue.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants