Conversation
Fixes #25
See also #79
There was a problem hiding this comment.
I didn't spot the changes here. Should be ok :)
There was a problem hiding this comment.
The config struct has 2 new fields
|
LGTM for me, but m server doesn't have SSL built in, so I didn't test this. If you tested it, I'm fine with it. |
|
Thought about it... |
There was a problem hiding this comment.
This if case is the only part that could break without being noticed by Travis. I don't think this is worth all the effort. I have 2 test systems with TLS enabled, this should be enough.
Fixes #25
Currently with Go 1.0 API in tests until Travis CI supports Go 1.1