Originally created by @bzed on GitHub (Oct 2, 2018).
Hi,
it would be great if you could pass all domains to the startup hook like it is done for the other hooks, or add an extra "validate_domains" hook. We need to check if all (customer) configured domains are avtually lvalid before trying to get certificates for them - right now its a script wrapped around dehydrated, but I'd prefer to be able to do this in a hook.
Thanks,
Bernd
Originally created by @bzed on GitHub (Oct 2, 2018).
Hi,
it would be great if you could pass all domains to the startup hook like it is done for the other hooks, or add an extra "validate_domains" hook. We need to check if all (customer) configured domains are avtually lvalid before trying to get certificates for them - right now its a script wrapped around dehydrated, but I'd prefer to be able to do this in a hook.
Thanks,
Bernd
Sorry, but I think that you should really validate the customer domains before handing them over to dehydrated.
The way startup_hook currently is implemented there is no way of adding the domains to it and your only move would be to cancel execution anyways, so it's not much better than a wrapper-script, actually even worse as with a wrapper you can use the --domain parameters and do your domain-list-parsing the way you need it.
Also you could set the KEEP_GOING parameter and dehydrated would just continue with the next certificate if there is an issue with domain validation.
@lukas2511 commented on GitHub (Oct 20, 2018):
Sorry, but I think that you should really validate the customer domains before handing them over to dehydrated.
The way `startup_hook` currently is implemented there is no way of adding the domains to it and your only move would be to cancel execution anyways, so it's not much better than a wrapper-script, actually even worse as with a wrapper you can use the `--domain` parameters and do your domain-list-parsing the way you need it.
Also you could set the `KEEP_GOING` parameter and dehydrated would just continue with the next certificate if there is an issue with domain validation.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Originally created by @bzed on GitHub (Oct 2, 2018).
Hi,
it would be great if you could pass all domains to the startup hook like it is done for the other hooks, or add an extra "validate_domains" hook. We need to check if all (customer) configured domains are avtually lvalid before trying to get certificates for them - right now its a script wrapped around dehydrated, but I'd prefer to be able to do this in a hook.
Thanks,
Bernd
@lukas2511 commented on GitHub (Oct 20, 2018):
Sorry, but I think that you should really validate the customer domains before handing them over to dehydrated.
The way
startup_hookcurrently is implemented there is no way of adding the domains to it and your only move would be to cancel execution anyways, so it's not much better than a wrapper-script, actually even worse as with a wrapper you can use the--domainparameters and do your domain-list-parsing the way you need it.Also you could set the
KEEP_GOINGparameter and dehydrated would just continue with the next certificate if there is an issue with domain validation.