From 4a259eda9cfe28e549909fb606879fd4aa4029fb Mon Sep 17 00:00:00 2001 From: Brad Warren Date: Mon, 27 Jul 2020 15:59:32 -0700 Subject: [PATCH] explain safe_copy --- certbot-apache/certbot_apache/_internal/apache_util.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/certbot-apache/certbot_apache/_internal/apache_util.py b/certbot-apache/certbot_apache/_internal/apache_util.py index 7196cbcfd..95029c5ad 100644 --- a/certbot-apache/certbot_apache/_internal/apache_util.py +++ b/certbot-apache/certbot_apache/_internal/apache_util.py @@ -74,7 +74,9 @@ def safe_copy(source, target): some uses of this function. After the permissions are applied, a check is done that the copied - file matches the source file. + file matches the source file. This original use for this behavior was for + Certbot to try and safely read the OCSP DBM cache file used by Apache when + we have no way synchronize reads and writes with the server. If any of this process fails, it will be retried twice before an exception is raised.