Log warnings in release

Kixunil · Aug 16, 2022 · 39bd78c · 39bd78c
1 parent ae9397e
commit 39bd78c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/bip78/Cargo.toml b/bip78/Cargo.toml
@@ -23,8 +23,8 @@ base64 = "0.13.0"
 rand = { version = "0.8.4", optional = true }
 bip21 = "0.1.1"
 url = "2.2.2"
+log = "0.4.14"
 
 [dev-dependencies]
 bitcoind = { version = "0.18.0", features = ["0_21_1"] }
 env_logger = "0.9.0"
-log = "0.4.14"
diff --git a/bip78/src/params.rs b/bip78/src/params.rs
@@ -45,6 +45,8 @@ impl Params {
                 ("additionalfeeoutputindex", index) =>  {
                     if let Ok(index) = index.parse::<usize>() {
                         additional_fee_output_index = Some(index);
+                    } else {
+                        log::warn!("bad `additionalfeeoutputindex` query value");
                     }
                 },
                 ("maxadditionalfeecontribution", fee) => {
@@ -60,8 +62,11 @@ impl Params {
                 _ => (),
             }
         }
-        if let (Some(amount), Some(index)) = (max_additional_fee_contribution, additional_fee_output_index) {
-            params.additional_fee_contribution = Some((amount, index));
+
+        match (max_additional_fee_contribution, additional_fee_output_index) {
+            (Some(amount), Some(index)) => params.additional_fee_contribution = Some((amount, index)),
+            (Some(_), None) | (None, Some(_)) => log::warn!("only one parameter specified"),
+            _ => ()
         }
 
         Ok(params)