Skip to content

TLS-implementation#2

Open
DigidragonZX wants to merge 3 commits intoCodeDrivenMitch:masterfrom
DigidragonZX:master
Open

TLS-implementation#2
DigidragonZX wants to merge 3 commits intoCodeDrivenMitch:masterfrom
DigidragonZX:master

Conversation

@DigidragonZX
Copy link
Copy Markdown

@DigidragonZX DigidragonZX commented Jan 10, 2019

I've created a fork that added TLS support for my own use, that could be merged if you agree with the changes.

The only thing that probably should be changed (I think made into an option) is the "rejectUnauthorized".

Christiaan Venhuizen added 3 commits December 21, 2018 12:12
…or now rejectUnauthorized is set to false since I'm using a

self-signed cert but this may become a setting in the future)
@DigidragonZX
Copy link
Copy Markdown
Author

I've never done a pull request so forgive me opening and closing it by accident.

On an other note I just saw that the AWS support was merged in, my code was based on the version prior to that so there are some merge conflicts.

@DigidragonZX DigidragonZX reopened this Jan 10, 2019
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

Successfully merging this pull request may close these issues.

1 participant