use --browser option

This commit is contained in:
Kevin Yue
2024-08-14 22:42:35 +08:00
parent a1c63f8498
commit f92b04e99a
3 changed files with 13 additions and 13 deletions

View File

@@ -41,7 +41,7 @@ struct Cli {
#[arg(long)]
default_browser: bool,
#[arg(long)]
external_browser: Option<String>,
browser: Option<String>,
}
impl Cli {
@@ -61,8 +61,8 @@ impl Cli {
None => portal_prelogin(&self.server, &gp_params).await?,
};
let browser_auth = if let Some(external_browser) = &self.external_browser {
Some(BrowserAuthenticator::new_with_browser(&saml_request, external_browser))
let browser_auth = if let Some(browser) = &self.browser {
Some(BrowserAuthenticator::new_with_browser(&saml_request, browser))
} else if self.default_browser {
Some(BrowserAuthenticator::new(&saml_request))
} else {

View File

@@ -102,7 +102,7 @@ pub(crate) struct ConnectArgs {
long,
help = "Use the specified browser to authenticate, e.g., firefox, chromium, chrome, or the path to the browser"
)]
external_browser: Option<String>,
browser: Option<String>,
}
impl ConnectArgs {
@@ -332,8 +332,8 @@ impl<'a> ConnectHandler<'a> {
match prelogin {
Prelogin::Saml(prelogin) => {
let use_default_browser = prelogin.support_default_browser() && self.args.default_browser;
let external_browser = if prelogin.support_default_browser() {
self.args.external_browser.as_deref()
let browser = if prelogin.support_default_browser() {
self.args.browser.as_deref()
} else {
None
};
@@ -349,7 +349,7 @@ impl<'a> ConnectHandler<'a> {
.ignore_tls_errors(self.shared_args.ignore_tls_errors)
.clean(self.args.clean)
.default_browser(use_default_browser)
.external_browser(external_browser)
.browser(browser)
.launch()
.await?;