From bea2ce76ba8587e80d59c16bf77e839ec7d4651b Mon Sep 17 00:00:00 2001
From: James Hodgkinson <james@terminaloutcomes.com>
Date: Thu, 22 Jul 2021 10:58:36 +1000
Subject: [PATCH] Relates to comments in #528 - pops error now instead of debug
 message (#529)

---
 kanidm_client/src/lib.rs | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/kanidm_client/src/lib.rs b/kanidm_client/src/lib.rs
index a6f50d21b..fbd846c71 100644
--- a/kanidm_client/src/lib.rs
+++ b/kanidm_client/src/lib.rs
@@ -144,7 +144,7 @@ impl KanidmClientBuilder {
         let address = match kcc.uri {
             Some(uri) => Some(uri),
             None => {
-                debug!("No URI in supplied config");
+                debug!("No URI in config supplied to apply_config_options");
                 address
             }
         };
@@ -285,17 +285,17 @@ impl KanidmClientBuilder {
     fn display_warnings(&self, address: &str) {
         // Check for problems now
         if !self.verify_ca {
-            warn!("verify_ca set to false - this may allow network interception of passwords!");
+            warn!("verify_ca set to false in client configuration - this may allow network interception of passwords!");
         }
 
         if !self.verify_hostnames {
             warn!(
-                "verify_hostnames set to false - this may allow network interception of passwords!"
+                "verify_hostnames set to false in client configuration - this may allow network interception of passwords!"
             );
         }
 
         if !address.starts_with("https://") {
-            warn!("address does not start with 'https://' - this may allow network interception of passwords!");
+            warn!("Address does not start with 'https://' - this may allow network interception of passwords!");
         }
     }
 
@@ -309,8 +309,8 @@ impl KanidmClientBuilder {
         let address = match &self.address {
             Some(a) => a.clone(),
             None => {
-                eprintln!("Configuration option 'uri' missing, cannot continue client startup.");
-                unimplemented!();
+                error!("Configuration option 'uri' missing from client configuration, cannot continue client startup without specifying a server to connect to. 🤔");
+                std::process::exit(1);
             }
         };