mirror of
https://github.com/dehydrated-io/dehydrated.git
synced 2026-01-11 22:30:44 +01:00
What am I doing wrong here? #252
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @TyrfingMjolnir on GitHub (Sep 12, 2017).
What am I doing wrong here?
@txr13 commented on GitHub (Sep 13, 2017):
To my eye, that looks like the challenge for domain.tld wasn't validated...
Can you please post the results of
dehydrated --versionfor us?@txr13 commented on GitHub (Sep 13, 2017):
@lukas2511 Looking through the code, I think we may have a bug?
In lines 606-610, we loop while
reqstatusis still pending. In lines 621-625, we check ifreqstatusis valid, and if not, we call the hook script... but we don't error out otherwise. Which means we move to validating the next SAN in the list.Once we're done with all the challenges, we move on. In line 635, we check again that
reqstatusis valid, but that's really only checking whether the last SAN validation succeeded.@TyrfingMjolnir commented on GitHub (Sep 18, 2017):
When I do ls in a different terminal on the folder
I can see 1 file named some random garble, it's gone when the script exits.
Here is the output from running dehydrate -c one more time.
( personal data neutralized below )
@TyrfingMjolnir commented on GitHub (Sep 18, 2017):
What I did wrong was to assign the location ".well-known/acme-challenge" to www.domain.tld opposed domain.tld.
@lukas2511 commented on GitHub (Sep 20, 2017):
@txr13 mh, you are right, that is a bug and i was able to reproduce it easily. i think a simple break should fix that, see
f838d93for my fix.it would also be possible to move the cleanup routine up, in that case it may even be simplified since it wouldn't need the loop over the invalid challenge files but only delete the current one, but maybe i'm forgetting something...