From aa8bdbde5afb1b8833a4a11d97bf4447db406437 Mon Sep 17 00:00:00 2001 From: Dan Mihai Date: Wed, 28 Aug 2024 02:26:08 +0000 Subject: [PATCH] agent: avoid policy.txt log without debug enabled slog's is_enabled() is documented as: - "best effort", and - Sometime resulting in false positives. Use AGENT_CONFIG.log_level.as_usize() instead, to avoid those false positives. Signed-off-by: Dan Mihai --- src/agent/src/policy.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/agent/src/policy.rs b/src/agent/src/policy.rs index 2416115853..d9ddba46dd 100644 --- a/src/agent/src/policy.rs +++ b/src/agent/src/policy.rs @@ -5,11 +5,10 @@ use anyhow::Result; use protobuf::MessageDyn; -use slog::Drain; use tokio::io::AsyncWriteExt; use crate::rpc::ttrpc_error; -use crate::AGENT_POLICY; +use crate::{AGENT_CONFIG, AGENT_POLICY}; static POLICY_LOG_FILE: &str = "/tmp/policy.txt"; @@ -87,7 +86,7 @@ impl AgentPolicy { /// Initialize regorus. pub async fn initialize(&mut self, default_policy_file: &str) -> Result<()> { - if sl!().is_enabled(slog::Level::Debug) { + if AGENT_CONFIG.log_level.as_usize() >= slog::Level::Debug.as_usize() { self.log_file = Some( tokio::fs::OpenOptions::new() .write(true)