This is an automated email from the ASF dual-hosted git repository.

bneradt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 0f50772212 Make OCSP refresh messages debug logs (#12329)
0f50772212 is described below

commit 0f507722125c60e34d463336ab414393996387ba
Author: Brian Neradt <[email protected]>
AuthorDate: Tue Jul 1 15:52:51 2025 -0500

    Make OCSP refresh messages debug logs (#12329)
    
    Boxes with OCSP configured typically have their logs containing many of
    these:
    
    ```
    $ tail diags.log
    [Jul  1 18:14:05.654] [ET_OCSP 0] NOTE: OCSP refresh started
    [Jul  1 18:14:05.654] [ET_OCSP 0] NOTE: OCSP refresh finished
    [Jul  1 18:15:05.649] [ET_OCSP 0] NOTE: OCSP refresh started
    [Jul  1 18:15:05.649] [ET_OCSP 0] NOTE: OCSP refresh finished
    [Jul  1 18:16:05.648] [ET_OCSP 0] NOTE: OCSP refresh started
    [Jul  1 18:16:05.648] [ET_OCSP 0] NOTE: OCSP refresh finished
    [Jul  1 18:17:05.644] [ET_OCSP 0] NOTE: OCSP refresh started
    [Jul  1 18:17:05.644] [ET_OCSP 0] NOTE: OCSP refresh finished
    [Jul  1 18:18:05.640] [ET_OCSP 0] NOTE: OCSP refresh started
    [Jul  1 18:18:05.640] [ET_OCSP 0] NOTE: OCSP refresh finished
    ```
    
    This is probably not generally helpful. There are error messages for
    OCSP failure scenarios. This patch downgrades these to debug logs.
---
 src/iocore/net/OCSPStapling.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/iocore/net/OCSPStapling.cc b/src/iocore/net/OCSPStapling.cc
index 7946c78129..af6d9f1819 100644
--- a/src/iocore/net/OCSPStapling.cc
+++ b/src/iocore/net/OCSPStapling.cc
@@ -1295,7 +1295,7 @@ ocsp_update()
   TS_OCSP_RESPONSE *resp = nullptr;
   time_t            current_time;
 
-  Note("OCSP refresh started");
+  Dbg(dbg_ctl_ssl_ocsp, "OCSP refresh started");
 
   SSLCertificateConfig::scoped_config certLookup;
 
@@ -1338,7 +1338,7 @@ ocsp_update()
       }
     }
   }
-  Note("OCSP refresh finished");
+  Dbg(dbg_ctl_ssl_ocsp, "OCSP refresh finished");
   return OCSPStatus::OCSP_OK;
 }
 

Reply via email to