From dd81d1d6e90c48eaf9df576352dba01f03d5c715 Mon Sep 17 00:00:00 2001 From: Patrick Donnelly Date: Tue, 27 May 2025 19:25:42 -0400 Subject: [PATCH] auth: cleanup error message formatting Signed-off-by: Patrick Donnelly --- src/auth/cephx/CephxSessionHandler.cc | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/auth/cephx/CephxSessionHandler.cc b/src/auth/cephx/CephxSessionHandler.cc index 0d6bff82432..f2545d1d5e8 100644 --- a/src/auth/cephx/CephxSessionHandler.cc +++ b/src/auth/cephx/CephxSessionHandler.cc @@ -192,13 +192,13 @@ int CephxSessionHandler::check_message_signature(Message *m) if (sig != m->get_footer().sig) { // Should have been signed, but signature check failed. PLR if (!(m->get_footer().flags & CEPH_MSG_FOOTER_SIGNED)) { - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << " Sender did not set CEPH_MSG_FOOTER_SIGNED." << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " Sender did not set CEPH_MSG_FOOTER_SIGNED." << dendl; } - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << " Message signature does not match contents." << dendl; - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << "Signature on message:" << dendl; - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << " sig: " << m->get_footer().sig << dendl; - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << "Locally calculated signature:" << dendl; - ldout(cct, 0) << "SIGN: MSG " << m->get_seq() << " sig_check:" << sig << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " Message signature does not match contents." << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " Signature on message:" << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " sig: " << m->get_footer().sig << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " Locally calculated signature:" << dendl; + ldout(cct, 0) << "SIGN: MSG seq " << m->get_seq() << " sig_check: " << sig << dendl; // For the moment, printing an error message to the log and // returning failure is sufficient. In the long term, we should -- 2.47.3