starred/acme-dns-client
1
0
Fork 0
mirror of https://github.com/acme-dns/acme-dns-client.git synced 2026-04-25 21:35:57 +03:00
Code Issues 20 Projects Releases 3 Packages Wiki Activity

[PR #3] [MERGED] Exit successfully if certbot renew validation is successful #23

New issue
Closed
opened 2026-03-13 16:27:08 +03:00 by kerem · 0 comments
kerem commented 2026-03-13 16:27:08 +03:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/acme-dns/acme-dns-client/pull/3
Author: @JonathanGawrych
Created: 2/16/2021
Status: ✅ Merged
Merged: 2/16/2021
Merged by: @joohoi

Base: master ← Head: jag/fix-certbot-renew

📝 Commits (1)

  • c44e83e Exit if certbot renew validation is successful

📊 Changes

1 file changed (+2 additions, -0 deletions)

View changed files

📝 main.go (+2 -0)

📄 Description

When certbot renew is run, acme-dns-client is called with 0 arguments. When validation for this passes, it doesn't exit with code 0. Instead it continues on, causing "index out of range [1]". This fix exits with 0 so that it doesn't throw later.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/acme-dns/acme-dns-client/pull/3 **Author:** [@JonathanGawrych](https://github.com/JonathanGawrych) **Created:** 2/16/2021 **Status:** ✅ Merged **Merged:** 2/16/2021 **Merged by:** [@joohoi](https://github.com/joohoi) **Base:** `master` ← **Head:** `jag/fix-certbot-renew` --- ### 📝 Commits (1) - [`c44e83e`](https://github.com/acme-dns/acme-dns-client/commit/c44e83e8ee3e29fc17e0e1ff28b1762ad64d6a07) Exit if certbot renew validation is successful ### 📊 Changes **1 file changed** (+2 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `main.go` (+2 -0) </details> ### 📄 Description When certbot renew is run, acme-dns-client is called with 0 arguments. When validation for this passes, it doesn't exit with code 0. Instead it continues on, causing "index out of range [1]". This fix exits with 0 so that it doesn't throw later. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-13 16:27:08 +03:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
release03
fix_wildcard
fix_goreleaser
prepare_v0.2
v0.3
v0.2
v0.1
Labels
Clear labels   
pull-request

Mirrored from GitHub Pull Request

  
question

   
question
No labels
pull-request
question
question
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/acme-dns-client#23
No description provided.