saltyrtc / saltyrtc-server-python

SaltyRTC signalling server implementation.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

s/SSL/TLS/

dbrgn opened this issue · comments

We should rename the --sslcert and --sslkey options to --tlscert and --tlskey. Similarly, the -sk and -tk options could be renamed too. SSL is dead :)

To provide backwards compatibility the options could still be supported, but not documented in the --help output.

commented

This is fine by me although my personal opinion is that SSL is synonymous to TLS today and I don't believe it will ever change, see for example: https://docs.python.org/3/library/ssl.html

I'm pretty sure we can add an alias option in click but I don't think there's a way to hide a specific one. I'd be fine with just an alias.

Python has reverse compatibility issues. We don't.

We can document the SSL versions as a deprecated alias of the TLS versions and remove them after a few releases.

commented

Again, I'm fine if we change it but I personally don't really care. 😅

Ok, I'll provide a PR sometime :)

commented

Closed due to inactivity.

commented

But, but, but… it's not "fixed". 😑

commented

The wontfix label should indicate that I'll accept PRs for this. It's just not going to be me who will create that PR.

I'll provide a PR.