wait.For() can return ambigous errors

[quenbyako]

• Yes, I'm using a binary release within 2 latest releases.
• Yes, I've searched similar issues on GitHub and didn't find any.
• Yes, I've included all information below (version, config, etc).

What did you expect to see?

wait.For returns nil error if deadline exceeded

What did you see instead?

If certificate got deadline For() returns nil error wrapped by fmt:

obtaining certificate: error: one or more domains had a problem:
[example.com] time limit exceeded: last error: %!w(<nil>)

So, this is not good: it's way more better if we can replace timeout to context and return DeadlineExceeded or bare error which returns from callback

Steps to reproduce

1. idk, just put into wait.For() some callback, which never returns any error

Details

Version of lego

discovered on v4.2.0 reproduces as well on v4.3.1

Logs

no any logs, idk

[quenbyako]

how fast lego developers are:

Thanks a lot for this fix! You rock guys, love this project