Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added support for client-side SSL #78

Merged
merged 3 commits into from Sep 1, 2012

Conversation

Projects
None yet
4 participants
Contributor

sharedinnovationlab commented Aug 31, 2012

Created a new security object delegate ClientSSLSecurity that exposes key/cert for consumption by request and https.

Owner

vpulim commented Sep 1, 2012

thanks, this looks great. one small issue before i merge: don't you need to add var https = require('https')? otherwise, i think new https.Agent() would throw an exception.

Contributor

jasonwilson commented Sep 1, 2012

Good eye! Must have been accidentally yanked before the commit somehow, all is working now.

Collaborator

milewise commented Sep 1, 2012

great! merging...

milewise added a commit that referenced this pull request Sep 1, 2012

Merge pull request #78 from nordstrom-innovation/master
Added support for client-side SSL

@milewise milewise merged commit 8dd03b0 into vpulim:master Sep 1, 2012

diarmaidm pushed a commit to diarmaidm/node-soap that referenced this pull request Feb 3, 2016

Merge pull request #78 from nordstrom-innovation/master
Added support for client-side SSL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment