Skip to content

Commit 246910c

Browse files
committed
fix: omit ExtraArgs from logs but add message to show when included
1 parent a2cd48f commit 246910c

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

Flow.Launcher/PublicAPIInstance.cs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -464,7 +464,11 @@ private void OpenUri(Uri uri, bool? inPrivate = null, bool forceBrowser = false)
464464
catch (Exception e)
465465
{
466466
var tabOrWindow = browserInfo.OpenInTab ? "tab" : "window";
467-
LogException(ClassName, $"Failed to open URL in browser {tabOrWindow}: {path}, {inPrivate ?? browserInfo.EnablePrivate}, {browserInfo.PrivateArg}, {browserInfo.ExtraArgs}", e);
467+
string includesExtraArgs = string.IsNullOrWhiteSpace(browserInfo.ExtraArgs)
468+
? ""
469+
: ", [including omitted Extra Args]";
470+
471+
LogException(ClassName, $"Failed to open URL in browser {tabOrWindow}: {path}, {inPrivate ?? browserInfo.EnablePrivate}, {browserInfo.PrivateArg}{includesExtraArgs}", e);
468472
ShowMsgError(
469473
Localize.errorTitle(),
470474
Localize.browserOpenError()

0 commit comments

Comments
 (0)