0ct0pu5/ffsend
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Report proper error for invalid API version argument

Browse source
This commit is contained in:
timvisee 2019-02-27 23:23:02 +01:00
parent 3bb0dc28dc
commit af98e133dc
No known key found for this signature in database
GPG key ID: B8DB720BC383E172
1 changed files with 5 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/cmd/arg/api.rs
View file

@ -3,6 +3,7 @@ use ffsend_api::api::{DesiredVersion, Version};
use super::{CmdArg, CmdArgOption};
use crate::config::API_VERSION_DESIRED_DEFAULT;
use crate::util::{ErrorHints, quit_error_msg};
/// The api argument.
pub struct ArgApi {}
@ -42,15 +43,10 @@ impl<'a> CmdArgOption<'a> for ArgApi {
// Parse the given API version
match Version::parse(version) {
Ok(version) => DesiredVersion::Use(version),
Err(_) => {
panic!("failed to determine given server API version, version unknown");
// TODO: properly quit with an application error instead
// quit_error(
// err.context("failed to determine given server API version, version unknown"),
// ErrorHints::default(),
// )
},
Err(_) => quit_error_msg(
"failed to determine given server API version, version unknown",
ErrorHints::default(),
),
}
}
}