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

letsencrypt: Add Namecheap DNS challenge support #3162

Merged
merged 7 commits into from
Dec 5, 2023

Conversation

nhurlock
Copy link
Contributor

@nhurlock nhurlock commented Aug 3, 2023

closes: #1368

@home-assistant
Copy link

home-assistant bot commented Aug 3, 2023

Hi @nhurlock

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link
Contributor

@mdegat01 mdegat01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change, looks good otherwise

letsencrypt/rootfs/etc/services.d/lets-encrypt/run Outdated Show resolved Hide resolved
@home-assistant home-assistant bot marked this pull request as draft September 1, 2023 15:24
@home-assistant
Copy link

home-assistant bot commented Sep 1, 2023

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@nhurlock nhurlock marked this pull request as ready for review September 2, 2023 02:22
@home-assistant home-assistant bot requested a review from mdegat01 September 2, 2023 02:22
@alistair23
Copy link

Gentle ping, this would be great to have

@Darsstar
Copy link

@mdegat01 This would be relly nice to have, and the change you requested seems to have been made.

@IGx89
Copy link

IGx89 commented Nov 6, 2023

Was just looking for this functionality myself. Would help me if this could be merged soon!

@agners
Copy link
Member

agners commented Dec 4, 2023

@nhurlock the add-on now got updated to the latest Alpine version etc. Can resolve the merge conflict. Then we should be able to merge this soon.

Copy link
Member

@agners agners left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor nits.

letsencrypt/Dockerfile Outdated Show resolved Hide resolved
letsencrypt/build.yaml Outdated Show resolved Hide resolved
letsencrypt/rootfs/etc/services.d/lets-encrypt/run Outdated Show resolved Hide resolved
@agners agners merged commit 5d0953e into home-assistant:master Dec 5, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: dns-namecheap support for LetsEncrypt
6 participants