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 Nginx from list of supported plugins #866

Closed
jsha opened this issue Sep 30, 2015 · 4 comments
Closed

Remove Nginx from list of supported plugins #866

jsha opened this issue Sep 30, 2015 · 4 comments

Comments

@jsha
Copy link
Contributor

jsha commented Sep 30, 2015

We have multiple reports that Nginx support is completely broken and breaks people's web servers when they run it. We should remove Nginx from the ncurses UI and from the command line help until it's fixed. Otherwise early beta participants will be very confused.

@centminmod
Copy link

i hope Nginx isn't removed from simplefs plugin #757 whenever it's merged as simplefs plugin works fine for my custom Nginx server where i auto generate the needed https before running the letsencrypt client in simplefs mode full example at https://community.centminmod.com/threads/letsencrypt-ssl-certificate-on-centmin-mod-nginx-http-2.4250/.

@bmw
Copy link
Member

bmw commented Oct 1, 2015

@centminmod, while I have other worries about the effectiveness of the SimpleFS plugin, it is server independent so you don't have to worry about Nginx support being removed there.

@kuba
Copy link
Contributor

kuba commented Oct 6, 2015

Rather than removing it from the UI, we should just make sure letsencrypt-nginx package is not installed in the user bootstrap scripts: this way it'll not register setuptools entry point and will not be present in the UI/help, but... if someone really wants to install it, it will appear in the UI / help. AKA: don't use hidden = True! :)

@bmw
Copy link
Member

bmw commented Oct 16, 2015

Kuba's suggestion for not installing letsencrypt-nginx was implemented here.

@bmw bmw closed this as completed Oct 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants