Honor the ssl config file & command-line option #90

Merged
merged 1 commit into from Sep 4, 2012

Projects

None yet

4 participants

@stash

Hello,

Thanks for making stud. Some time around merging @bfg's config file change (which is otherwise awesome) it seems the "ssl" config directive wasn't working. The --ssl command-line option also stopped working.

As you can see, the fix is trivial. My company has been using this fix in production for a few weeks (plus my smartos branch which has some fixes that should also work for Solaris in general).

Without this fix, Safari using wss:// cannot connect (something in the way the negotiation happens, I haven't been able to investigate it fully).

Cheers, and thanks in advance.

~stash

@bfg

OMG, what a silly typo, i am ashamed...

LOL, happens to the best of us. Having a config file has proven quite useful for how we have to deploy stud, so thanks for that. :)

@Roguelazer

...why is this not merged in yet?

@jamwt jamwt merged commit 19a7f19 into bumptech:master Sep 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment