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

2.9.0 release ? #71

Open
Deishelon opened this issue Mar 11, 2024 · 2 comments
Open

2.9.0 release ? #71

Deishelon opened this issue Mar 11, 2024 · 2 comments

Comments

@Deishelon
Copy link

Deishelon commented Mar 11, 2024

Hi,

If one is using nginx-proxy-manager, it will try to install this package, however using latest nginx-proxy-manager (2.11.1) with certbot 2.9.0 will fail with No matching distribution found for certbot-dns-godaddy==2.9.0

Looking at sources of nginx-proxy-manager, it's gonna install certbot-dns-godaddy==${certbot-version}. Ref

The way it gets certbot-version is by doing certbot --version Ref

Doing it on the latest nginx-proxy-manager, I get this:

$ docker exec -it 5042b6b961c1 /bin/bash

Version 2.11.1 (aec3020) 2024-02-28 22:33:06 UTC, OpenResty 1.21.4.3, debian 12 (bookworm), Certbot certbot 2.9.0
Base: debian:bookworm-slim, linux/amd64
Certbot: nginxproxymanager/nginx-full:latest, linux/amd64
Node: nginxproxymanager/nginx-full:certbot, linux/amd64

[root@docker-5042b6b961c1:/app]# certbot --version
certbot 2.9.0
[root@docker-5042b6b961c1:/app]#

So I think we effectively need release of this repo with version 2.9.0, or we open an issue in nginx-proxy-manager to use 2.8.0 (latest version of this repo at the time of writing)

Update: I should have checked nginx-proxy-manager repo for this issue. and indeed there is one: NginxProxyManager/nginx-proxy-manager#3592

But I think the proposed solutions there (exec into container, and hack stuff is not a great solution, so opening a issue here to fix the root cause)

@wolviex
Copy link

wolviex commented Mar 19, 2024

I think that the root cause is that NPM has tied the version of this plugin to the version of Certbot. This will come up every single time there is an update to certbot.
I've submitted a PR there to fix the root cause and pin NPM to use the latest certbot-dns-godaddy (2.8.0)
NginxProxyManager/nginx-proxy-manager#3639

I've encountered another error, relating to certbot/plugins/dns_common_lexicon.py and the recent update about delegated subdomains, so try at your own risk.

@miigotu
Copy link
Owner

miigotu commented Mar 25, 2024

I think that the root cause is that NPM has tied the version of this plugin to the version of Certbot. This will come up every single time there is an update to certbot.
I've submitted a PR there to fix the root cause and pin NPM to use the latest certbot-dns-godaddy (2.8.0)
NginxProxyManager/nginx-proxy-manager#3639

I've encountered another error, relating to certbot/plugins/dns_common_lexicon.py and the recent update about delegated subdomains, so try at your own risk.

We need a workflow or bot here that automatically updates this package when certbot is updated, and tests it and then releases if someone wants to take that on.

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