-
Notifications
You must be signed in to change notification settings - Fork 119
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
Add additional Tls/SSL arguments #275
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good! I've made some doc suggestions for consistency, but otherwise looks great.
I've allowed workflows for this PR so you can check linting and tests. You'll also need to resolve some conflicts on the readme. Nice work though, so I'll be happy to merge this after these tweaks! ❤️ |
Co-authored-by: Dave Hall <[email protected]>
Co-authored-by: Dave Hall <[email protected]>
Co-authored-by: Dave Hall <[email protected]>
Co-authored-by: Dave Hall <[email protected]>
Thanks! I've applied the changes now. |
Merged, and will shortly be released! |
Published as 0.15.8 |
I'm running an LDAP server with a handful of old/legacy settings required, which causes this to fail to connect. Pullrequest adds support for entering those settings, and support for other unnamed options through keyword arguments.