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
certbot: 1.3.0 -> 1.6.0 #92696
certbot: 1.3.0 -> 1.6.0 #92696
Conversation
This includes certbot/certbot#8081, so we can stop shipping our custom patch to workaroung things, and start packaging more certbot plugins. Other release notes: - https://github.com/certbot/certbot/releases/tag/v1.4.0 - https://github.com/certbot/certbot/releases/tag/v1.5.0 - https://github.com/certbot/certbot/releases/tag/v1.6.0
I added 3 common DNS plugins I might end up using - the others should be simple enough to be packaged once someone starts using them. I was able to access these plugins from a @FRidh, @domenkozar, do you have any opinions on a |
cc @NixOS/acme |
Is the intention long-term to replace lego with cerbot? If so, I might just do it during my rewrite. |
This is unrelated to the acme module system - I have some different tooling currently hard-wired to certbot and using some of the plugins packaged here - I still thought it might make sense to highlight the acme team.
|
We probably want to remove these packages from the python2 packageset. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Portability nitpick.
Other than that, LGTM
pkgs/development/python-modules/certbot-dns-cloudflare/default.nix
Outdated
Show resolved
Hide resolved
pkgs/development/python-modules/certbot-dns-rfc2136/default.nix
Outdated
Show resolved
Hide resolved
pkgs/development/python-modules/certbot-dns-route53/default.nix
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@GrahamcOfBorg build certbot |
This includes certbot/certbot#8081, so we can
stop shipping our custom patch to workaroung things, and start packaging
more certbot plugins.
Other release notes:
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)