this also gates the tracing logs in axum under a tracing feature, i can nix this if need be
+1
-4
crates/jacquard/src/client/bff_session.rs
+1
-4
crates/jacquard/src/client/bff_session.rs
+1
crates/jacquard-axum/Cargo.toml
+1
crates/jacquard-axum/Cargo.toml
+14
-7
crates/jacquard-axum/src/service_auth.rs
+14
-7
crates/jacquard-axum/src/service_auth.rs
···
572
573
match codec {
574
// p256-pub (0x1200)
575
-
[0x80, 0x24] => PublicKey::from_p256_bytes(key_material).inspect_err(|e| {
576
-
tracing::error!("Failed to parse p256 public key: {}", e);
577
-
}).ok(),
578
// secp256k1-pub (0xe7)
579
-
[0xe7, 0x01] => PublicKey::from_k256_bytes(key_material).inspect_err(|e| {
580
-
tracing::error!("Failed to parse secp256k1 public key: {}", e);
581
-
}).ok(),
582
_ => {
583
tracing::error!("Unsupported public key multicodec: {:?}", codec);
584
None
585
-
},
586
}
587
}
588
···
572
573
match codec {
574
// p256-pub (0x1200)
575
+
[0x80, 0x24] => PublicKey::from_p256_bytes(key_material)
576
+
.inspect_err(|_e| {
577
+
#[cfg(feature = "tracing")]
578
+
tracing::error!("Failed to parse p256 public key: {}", _e);
579
+
})
580
+
.ok(),
581
// secp256k1-pub (0xe7)
582
+
[0xe7, 0x01] => PublicKey::from_k256_bytes(key_material)
583
+
.inspect_err(|_e| {
584
+
#[cfg(feature = "tracing")]
585
+
tracing::error!("Failed to parse secp256k1 public key: {}", _e);
586
+
})
587
+
.ok(),
588
_ => {
589
+
#[cfg(feature = "tracing")]
590
tracing::error!("Unsupported public key multicodec: {:?}", codec);
591
None
592
+
}
593
}
594
}
595
-2
crates/jacquard-oauth/src/client.rs
-2
crates/jacquard-oauth/src/client.rs
-5
crates/jacquard-oauth/src/request.rs
-5
crates/jacquard-oauth/src/request.rs
History
1 round
2 comments
nekomimi.pet
submitted
#0
expand 2 comments
i did actually merge this, i think tangled just got confused with the jujutsu rebase on top of your merge i did
closed without merging
crap, meant to pull this into the release, will do another version bump in a bit, this is good overall. need to take a real look over logging/errors again.