🔊 Use tracing::warn!() not eprintln!()

dbus2#159 (comment)
jokeyrhyme · Nov 15, 2024 · fa0c5d5 · fa0c5d5
1 parent 74fba26
commit fa0c5d5
Showing 1 changed file with 10 additions and 13 deletions.
diff --git a/src/config.rs b/src/config.rs
@@ -6,6 +6,7 @@ use serde::Deserialize;
 
 mod xml;
 
+use tracing::warn;
 use xml::{
     Document, Element, PolicyContext, PolicyElement, RuleAttributes, RuleElement, TypeElement,
 };
@@ -109,7 +110,7 @@ impl TryFrom<Document> for BusConfig {
                 Element::Fork => bc.fork = true,
                 Element::KeepUmask => bc.keep_umask = true,
                 Element::Limit => {
-                    eprintln!("warning: busd does not implement `<limit>`");
+                    warn!("warning: busd does not implement `<limit>`");
                 }
                 Element::Listen(s) => {
                     bc.listen.insert(s);
@@ -355,14 +356,10 @@ impl TryFrom<RuleAttributes> for OptionalOperation {
 
         // https://github.com/dbus2/busd/issues/79
         if value.receive_member.is_some() {
-            eprintln!(
-                "warning: busd does not implement `<(allow|deny) receive_member=...`: {value:?}"
-            );
+            warn!("warning: busd does not implement `<(allow|deny) receive_member=...`: {value:?}");
         }
         if value.send_member.is_some() {
-            eprintln!(
-                "warning: busd does not implement `<(allow|deny) send_member=...`: {value:?}"
-            );
+            warn!("warning: busd does not implement `<(allow|deny) send_member=...`: {value:?}");
         }
 
         if has_connect {
@@ -401,7 +398,7 @@ impl TryFrom<PolicyElement> for OptionalPolicy {
                 user: None,
                 ..
             } => {
-                eprintln!(r#"warning: busd does not implement `<policy at_context="...">`"#);
+                warn!(r#"warning: busd does not implement `<policy at_context="...">`"#);
                 Ok(None)
             }
             PolicyElement {
@@ -507,7 +504,7 @@ impl TryFrom<RuleElement> for OptionalRule {
                 ..
             }) => {
                 // see: https://github.com/dbus2/busd/pull/146#issuecomment-2408429760
-                eprintln!(
+                warn!(
                     r#"warning: eavesdropping is deprecated and ignored: `<allow eavesdrop="true" />`"#
                 );
                 Ok(None)
@@ -523,15 +520,15 @@ impl TryFrom<RuleElement> for OptionalRule {
                 },
             ) => {
                 // see: https://github.com/dbus2/busd/pull/146#issuecomment-2408429760
-                eprintln!(
+                warn!(
                     "warning: explicit policies on replies and errors are deprecated and ignored: `<allow ... (receive|send)_requested_reply=...`"
                 );
                 Ok(None)
             }
             RuleElement::Allow(attrs) => {
                 if attrs.eavesdrop == Some(true) {
                     // see: https://github.com/dbus2/busd/pull/146#issuecomment-2408429760
-                    eprintln!(
+                    warn!(
                         r#"warning: eavesdropping is deprecated and ignored: `<allow ... eavesdrop="true" ...`"#
                     );
                 }
@@ -545,7 +542,7 @@ impl TryFrom<RuleElement> for OptionalRule {
                 ..
             }) => {
                 // see: https://github.com/dbus2/busd/pull/146#issuecomment-2408429760
-                eprintln!(
+                warn!(
                     r#"warning: eavesdropping is deprecated and ignored: `<deny ... eavesdrop="true" ...`"#
                 );
                 Ok(None)
@@ -561,7 +558,7 @@ impl TryFrom<RuleElement> for OptionalRule {
                 },
             ) => {
                 // see: https://github.com/dbus2/busd/pull/146#issuecomment-2408429760
-                eprintln!(
+                warn!(
                     "warning: explicit policies on replies and errors are deprecated and ignored: `<deny ... (receive|send)_requested_reply=...`"
                 );
                 Ok(None)