fix: linting, cargo-deny

Author: varex83

Cargo.toml

@@ -56,7 +56,6 @@ charon-eth2 = { path = "crates/charon-eth2" }
 charon-k1util = { path = "crates/charon-k1util" }
 charon-p2p = { path = "crates/charon-p2p" }
 charon-testutil = { path = "crates/charon-testutil" }
-charon-tracing = { path = "crates/charon-tracing" }
 
 [workspace.lints.rust]
 missing_docs = "deny"

crates/tracing/examples/basic.rs

@@ -33,15 +33,21 @@ async fn main() {
 
     let background_task = init(&config)
         .expect("Failed to initialize tracing")
-        .unwrap();
+        .expect("Background task should be Some");
 
     tokio::spawn(background_task);
 
     let bind_address = SocketAddr::from(([0, 0, 0, 0], 9464));
 
-    let exporter = MetricsExporter::default().bind(bind_address).await.unwrap();
+    let exporter = MetricsExporter::default()
+        .bind(bind_address)
+        .await
+        .expect("Failed to bind metrics exporter");
     tokio::spawn(async move {
-        exporter.start().await.unwrap();
+        exporter
+            .start()
+            .await
+            .expect("Failed to start metrics exporter");
     });
 
     // Test various log levels

crates/tracing/src/layers/metrics.rs

@@ -18,11 +18,11 @@ impl<S: tracing::Subscriber> tracing_subscriber::Layer<S> for MetricsLayer {
         _ctx: tracing_subscriber::layer::Context<'_, S>,
     ) {
         // check level
-        match event.metadata().level() {
-            &tracing::Level::ERROR => {
+        match *event.metadata().level() {
+            tracing::Level::ERROR => {
                 inc_error_count();
             }
-            &tracing::Level::WARN => {
+            tracing::Level::WARN => {
                 inc_warn_count();
             }
             _ => {