cert-manager/pkg/issuer/acme
Maxim Ivanov c44a7552ea Check challenge before presenting it
With async challenge Check, it is often happens,
that solver.Check() fails on first run after solver.Present()

Cert-manager then tries again, but starts with solver.Present(),
which not being idempotent right now fails on certain DNS providers.

This change swaps order of solver.Check() and solver.Present().
Check is not returning error if propagation not happened, it then
allows Present() to run.

In the current form, Present() will be spamming with errors,
but this doesn't stop Check from happening on every attempt,
so eventually Challenge can be verified and accepted. In the future,
Present() should be made idempotent.
2018-04-11 11:27:23 +01:00
..
client Fix infinite loop in logger middleware 2018-04-09 19:09:46 +01:00
dns Check challenge before presenting it 2018-04-11 11:27:23 +01:00
http Enable DNS01 provider tests using cloudflare 2018-04-10 00:27:52 +01:00
acme.go Check challenge before presenting it 2018-04-11 11:27:23 +01:00
issue.go Remove unused strings and standardise event reasons 2018-04-09 21:26:38 +01:00
prepare_test.go Rewrite ACME issuer to use new ACMEOrderChallenge struct 2018-04-09 15:40:32 +01:00
prepare.go Check challenge before presenting it 2018-04-11 11:27:23 +01:00
renew.go Improve error reporting and use of status conditions 2018-04-09 21:17:51 +01:00
setup.go Use GetAccount to check if acme account is already registered 2018-03-23 18:50:46 +00:00
util_test.go Fix up bugs in unit testing framework 2018-04-04 23:40:44 +01:00
util.go Rewrite ACME issuer to use new ACMEOrderChallenge struct 2018-04-09 15:40:32 +01:00