-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Output of external commands gets printed to stdout #20
Comments
Makes sense. Probably better with a flag (defaulted to false) that can be enabled to switch on the output. |
Workaround: Add to your Cargo.toml
In your code: use webbrowser::{self, BrowserOptions};
if webbrowser::open_browser_with_options(
BrowserOptions {
url: String::from(token_url),
suppress_output: Some(true),
browser: Some(webbrowser::Browser::Default),
}).is_ok() {} |
Ultimately should merge #22 though |
Hello! I am having the same issue here, bump ? :) I can take care of it if no one is doing it |
Fixed in ecfbf66. By default, we now suppress output for all GUI browsers. This behaviour can be overridden. See Consistent Behaviour section for details. |
In my case on Linux the call to
xdg-open
causesOpening in existing browser session
to be printed to the console. Would it make sense to silence all output of the external tools by default unless there's an error?The text was updated successfully, but these errors were encountered: