Relates to comments in #528 - pops error now instead of debug message (#529)

This commit is contained in:
James Hodgkinson 2021-07-22 10:58:36 +10:00 committed by GitHub
parent ff952cbed2
commit bea2ce76ba
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -144,7 +144,7 @@ impl KanidmClientBuilder {
let address = match kcc.uri { let address = match kcc.uri {
Some(uri) => Some(uri), Some(uri) => Some(uri),
None => { None => {
debug!("No URI in supplied config"); debug!("No URI in config supplied to apply_config_options");
address address
} }
}; };
@ -285,17 +285,17 @@ impl KanidmClientBuilder {
fn display_warnings(&self, address: &str) { fn display_warnings(&self, address: &str) {
// Check for problems now // Check for problems now
if !self.verify_ca { 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 { if !self.verify_hostnames {
warn!( 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://") { 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 { let address = match &self.address {
Some(a) => a.clone(), Some(a) => a.clone(),
None => { None => {
eprintln!("Configuration option 'uri' missing, cannot continue client startup."); error!("Configuration option 'uri' missing from client configuration, cannot continue client startup without specifying a server to connect to. 🤔");
unimplemented!(); std::process::exit(1);
} }
}; };