Karl Tarbe comments:
Anyway, the error message should be improved: instead of saying "Certificate check failed!", I would expect to see a reason why the check failed. In this case, that reason would have been "Certificate is expired!". Actually when testing, I even wondered, if the certificate might have been expired. In the code there was something like: try: blah(), expect: LOG.debug("Certificate check failed!"). This is catch em all, which is bad style and the real reason, why the check failed is not reported.
Karl Tarbe comments:
Anyway, the error message should be improved: instead of saying "Certificate check failed!", I would expect to see a reason why the check failed. In this case, that reason would have been "Certificate is expired!". Actually when testing, I even wondered, if the certificate might have been expired. In the code there was something like: try: blah(), expect: LOG.debug("Certificate check failed!"). This is catch em all, which is bad style and the real reason, why the check failed is not reported.