Skip to content

Commit d857124

Browse files
feat(logs): adding extra logs
1 parent 87379a1 commit d857124

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

agent-control/src/opamp/remote_config/validators/signature/certificate.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
use ring::digest;
22
use std::fmt::Write;
33
use thiserror::Error;
4+
use tracing::trace;
45
use webpki::EndEntityCert;
56
use x509_parser::prelude::{FromDer, X509Certificate};
67

@@ -40,7 +41,10 @@ impl Verifier for Certificate {
4041

4142
certificate
4243
.verify_signature(signature_algorithm, msg, signature)
43-
.map_err(|e| CertificateError::VerifySignature(e.to_string()))
44+
.map_err(|e| CertificateError::VerifySignature(e.to_string()))?;
45+
trace!("signature verification succeeded");
46+
47+
Ok(())
4448
}
4549

4650
fn key_id(&self) -> &str {

agent-control/src/opamp/remote_config/validators/signature/public_key.rs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ use base64::{Engine, prelude::BASE64_STANDARD};
66
use ring::digest;
77
use ring::signature::{ED25519, UnparsedPublicKey};
88
use thiserror::Error;
9+
use tracing::trace;
910

1011
#[derive(Error, Debug)]
1112
pub enum PubKeyError {
@@ -81,7 +82,10 @@ impl Verifier for PublicKey {
8182
.verify(msg.as_bytes(), signature)
8283
.map_err(|_| {
8384
PubKeyError::ValidatingSignature("signature verification failed".to_string())
84-
})
85+
})?;
86+
trace!(%self.key_id, "signature verification succeeded");
87+
88+
Ok(())
8589
}
8690

8791
fn key_id(&self) -> &str {

0 commit comments

Comments
 (0)