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

ACME Renewal Information (ARI) #403

Closed
mholt opened this issue Mar 27, 2023 · 8 comments · Fixed by #461
Closed

ACME Renewal Information (ARI) #403

mholt opened this issue Mar 27, 2023 · 8 comments · Fixed by #461

Comments

@mholt
Copy link

mholt commented Mar 27, 2023

Hi -- does pebble support ARI yet? I did a search but couldn't find any commits or issues referring to it. I'm starting to implement it into acmez/CertMagic/Caddy, but if it's not in pebble yet I'm just wondering if I should test against LE staging for now?

Thanks!

@jvanasco
Copy link
Contributor

jvanasco commented May 2, 2024

Pebble still doesn't have ARI.

Certbot recently dropped Boulder in favor of Pebble as the testing mechanism - See certbot/certbot#9918

Given that new utilization by the largest ACME client, I think we can consider the lack of ARI functionality by Pebble as a roadblock widespread adoption.

[I would code a PR, but my Go knowledge is limited and I don't have enough bandwidth to expand my capabilities here to make a quality PR]

@pgporada
Copy link
Member

pgporada commented May 2, 2024

I've got a work in progress started here.

@jvanasco
Copy link
Contributor

jvanasco commented May 2, 2024

@pgporada I you. Thank you! I'll be monitoring that PR so I can better understand how this works and (hopefully) provide quality PRs in the future.

@mholt
Copy link
Author

mholt commented May 2, 2024

This is great, thank you. As soon as that's ready I'll prioritize implementing ARI.

@pgporada
Copy link
Member

pgporada commented May 24, 2024

@mholt @jvanasco ARI support has landed as of #461. Let me know how it works out for you or if I've introduced any fun and interesting bugs.

@mholt
Copy link
Author

mholt commented May 24, 2024

Hurray! Thanks Phil!

@felixfontein
Copy link
Contributor

Thanks a lot @pgporada for implementing this, and @aarongable and @beautifulentropy for reviewing! I'm looking forward to try this out :)

@pgporada
Copy link
Member

pgporada commented May 24, 2024

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

Successfully merging a pull request may close this issue.

4 participants