Consider adding SSL support #3

wants to merge 2 commits into

6 participants


Hey there,

I've added SSL support, and it is a very minor change, so I think it would be beneficial if you merge this into the upstream :)


Brendan Molloy added some commits Sep 12, 2010

Thanks for providing this, I just put this in my copy of Phenny. Would really be nice if this was in the main release.


Considering how simple the change is, yes, it would be. Does it still apply cleanly to mainline?


I downloaded a fresh copy of the yesterday and applied these changes to get it to connect to our SSL only server. Worked fine!


Thank you bbqsrc! Works great.


Would you mind squashing the commits so that I can pull it? Thanks!


I did a git clone on bbqsrc's phenny repository and used the following in the

host = ''
port = 7000
ssl = True

and phenny disconnects with:

Connecting to connected!
Warning: Disconnected. Reconnecting in 20 seconds...

I'm afraid this doesn't work for me either. Same error as above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment