chore(rust): fix typo and re-add instrumentation

fix typo on comment
build-trust · May 25, 2024 · 7ab0742 · 7ab0742
1 parent 2d69a69
commit 7ab0742
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/implementations/rust/ockam/ockam_abac/src/policy/policies.rs b/implementations/rust/ockam/ockam_abac/src/policy/policies.rs
@@ -9,7 +9,7 @@ use ockam_core::compat::vec::Vec;
 use ockam_core::Result;
 use ockam_identity::{Identifier, IdentitiesAttributes};
 use strum::IntoEnumIterator;
-use tracing::debug;
+use tracing::{debug, instrument};
 
 #[derive(Clone)]
 pub struct Policies {
@@ -28,7 +28,7 @@ impl Policies {
         }
     }
 
-    //TODO #[instrument(skip_all, fields(resource = %resource, action = %action, env = %env, authority = %authority))]
+    #[instrument(skip_all, fields(resource = %resource, action = %action, env = %env, authority = ?authority))]
     pub fn make_policy_access_control(
         &self,
         identities_attributes: Arc<IdentitiesAttributes>,

diff --git a/implementations/rust/ockam/ockam_api/src/nodes/service/manager.rs b/implementations/rust/ockam/ockam_api/src/nodes/service/manager.rs
@@ -412,7 +412,7 @@ impl NodeManager {
             }
         } else {
             // If no expression is given, assume it's AllowAll, but only if no authority
-            // was set neither. Why: not sure, but to behave as it was previusly if there
+            // was set neither. Why: not sure, but to behave as it was previously if there
             // is an authority set.  If there is no authority, but still some expression,
             // we use the provided policy expression
             warn! {