Skip to content
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

Remove --disable-tls ? #834

Open
vanosg opened this issue Aug 5, 2019 · 0 comments

Comments

@vanosg
Copy link
Member

commented Aug 5, 2019

The original SSL/TLS code was added to Eggdrop in 2010- nearly a decade ago. A lot has changed since then! When first introduced, TLS was not as common, or known to be as necessary, as it is now. Users were offered the option to disable TLS due to issues connecting with other botnets in certain circumstances, as well as not every machine having TLS dev libs installed to compile against- hence Eggdrop's ability to disable TLS functionality if the TLS libs were not detected, or a user intentionally disabled them with --disable-tls during configure. TLS is not required for Eggdrop to connect to an IRC server and function in a near-full capacity, and users may not have the permissions needed to install the TLS dev libraries on a machine.

As TLS adaption has become more prevalent, and as the Eggdrop TLS codebase has stabilized, has the time come to remove the option for --disable-tls ? I want to be clear, this is not removing the ability for eggdrop to compile without TLS libs, but if they are present on the system, to require their usage?

I'd like to open this up for discussion- I don't want to force something on the users because 'we' think it is better for them, but at the same time I think the argument for enabling a manual disabling of TLS is starting to lose some of its strength. What do you think? Is it time to remove --disable-tls ?

@vanosg vanosg added this to the v1.9.0 milestone Aug 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.