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

Recommendation to Prioritize Snap Installation for Certbot and Deprecate Outdated Install Methods #206

Open
onknows opened this issue Jan 12, 2024 · 2 comments

Comments

@onknows
Copy link

onknows commented Jan 12, 2024

I've noticed that the geerlingguy.certbot Ansible role currently supports package and source installation methods for Certbot. However, I propose reconsidering these options due to their limitations and potential misalignment's with the Certbot community's recommendations.

Package Install Method Concerns: When using the package install method, especially on platforms like Ubuntu 22, users often end up with outdated versions of Certbot. The Certbot community predominantly supports the Snap installation method. While third-party packages might be available in some scenarios, having Snap as the default installation method seems more aligned with current best practices.

Source Install Method Issues: The source installation approach relies on the certbot-auto script, which is no longer available or supported. This method fails to work as intended. An alternative could be to develop a custom script within this role, but it's crucial to assess its maintenance and support implications.

Given these considerations, I suggest:

  1. Making Snap the default installation method.
  2. Adding warnings or deprecating the package and source install methods.
  3. Investigating the feasibility of incorporating a custom script for source installation, if deemed necessary.

These changes should help ensure users have the most reliable and up-to-date Certbot experience.

@C0rn3j
Copy link

C0rn3j commented Apr 16, 2024

If you mind your repository package is out of date, package a newer version yourself or use a third party repository, it's not in the scope of the role to fix that for you.

I also disagree with tying this role to Canonical-controlled backend and even deprecating package installs, that's just bad, despite LE thinking it's a good idea.
The role DOES support Snap, it's just not default - https://github.com/geerlingguy/ansible-role-certbot?tab=readme-ov-file#snap-installation
.
If the current source scripts are broken, it would be nice to fix them.
Seeing that upstream deprecated the previous method, that one seems fun to do.

The most sane option would be to install the latest version in a venv.

https://eff-certbot.readthedocs.io/en/latest/install.html#alternative-2-pip

Notably, by NOT following the upstream documentation, which seems to suggest you wreck your system packages to get it working by means of sudo pip, which won't even work on modern distributions that enabled a safeguard against this.

@funkyfuture
Copy link

looking at the current instructions selection form for Certbot it's obvious that the vendor doesn't share the resentment towards Snap as many other people. so my conclusion would be that the snap based deployment should be default for platforms that aren't supported otherwise, e.g. Ubuntu 22+.

a pipx based deployment option seems reasonable and interesting to me.

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

3 participants