Skip to content

Commit

Permalink
(PE-37635) adjust logging of error to error level, add info logging
Browse files Browse the repository at this point in the history
This adjusts the logging of failures during bulk cert signing to error
so they end up in the standard log. Additionally, this adds info level
logging of the subject name for a newly signed cert which is done
on the old endpoint.
  • Loading branch information
jonathannewman committed Feb 9, 2024
1 parent 6100a56 commit d21cf85
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/clj/puppetlabs/puppetserver/certificate_authority.clj
Original file line number Diff line number Diff line change
Expand Up @@ -2465,10 +2465,11 @@
(write-cert-to-inventory-unlocked! signed-cert ca-settings)
(write-cert signed-cert (path-to-cert signeddir subject))
(delete-certificate-request! ca-settings subject)
(log/info (i18n/trs "Signed CSR for {0}" subject))
;; success case, add the host to the set of signed results
:signed))
(catch Throwable e
(log/debug e (i18n/trs "Failed in bulk signing for entry {0}" subject))
(log/error e (i18n/trs "Failed in bulk signing for entry {0}" subject))
;; failure case, add the host to the set of not signed results
:signing-errors)))

Expand Down

0 comments on commit d21cf85

Please sign in to comment.