Skip to content
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

Asking to open in browser that is not installed fails silently #421

Closed
andreamah opened this issue Jan 23, 2023 · 7 comments · Fixed by #515
Closed

Asking to open in browser that is not installed fails silently #421

andreamah opened this issue Jan 23, 2023 · 7 comments · Fixed by #515
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@andreamah
Copy link
Contributor

Repro:

  1. Set livePreview.customExternalBrowser to Firefox on a computer that doesn't have firefox installed
  2. Try Live Preview: Show Preview (External Browser)
  3. Fails silently 🐛
@andreamah andreamah added this to the Backlog milestone Jan 23, 2023
@andreamah andreamah self-assigned this Jan 23, 2023
@andreamah andreamah added the bug Issue identified by VS Code Team member as probable bug label Jan 23, 2023
@andreamah andreamah modified the milestones: Backlog, June 2023 Jun 12, 2023
andreamah added a commit that referenced this issue Jun 13, 2023
* Asking to open in browser that is not installed fails silently
Fixes #421
@rzhao271 rzhao271 added the verified Verification succeeded label Jun 28, 2023
@rzhao271
Copy link

rzhao271 commented Jun 28, 2023

I don't see any error, nor do I see any failure message.
I switched to the pre-release version beforehand.

@rzhao271 rzhao271 reopened this Jun 28, 2023
@rzhao271 rzhao271 added verification-found Issue verification failed and removed verified Verification succeeded labels Jun 28, 2023
@andreamah
Copy link
Contributor Author

Did you see a notif like
image
?

@rzhao271
Copy link

No. It just adds a "Port: 3000" entry to the status bar. Nothing else seems to happen

@andreamah
Copy link
Contributor Author

which version of ms-vscode.js-debug-companion are you running? are there any errors in devtools?

@andreamah andreamah modified the milestones: June 2023, July 2023 Jun 28, 2023
@rzhao271
Copy link

I get the following devtools error message on Windows 11.
I uninstalled Firefox before attempting to verify this issue.

mainThreadExtensionService.ts:81 [ms-vscode.js-debug-companion]Cannot read properties of undefined (reading 'Stable')
$onExtensionRuntimeError @ mainThreadExtensionService.ts:81
N @ rpcProtocol.ts:455
M @ rpcProtocol.ts:440
H @ rpcProtocol.ts:370
G @ rpcProtocol.ts:296
(anonymous) @ rpcProtocol.ts:161
w @ event.ts:1088
fire @ event.ts:1119
fire @ ipc.net.ts:650
Q.onmessage @ localProcessExtensionHost.ts:583
mainThreadExtensionService.ts:82 TypeError: Cannot read properties of undefined (reading 'Stable')
    at Rt.<anonymous> (c:\Users\raymondzhao\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\extensions\ms-vscode.js-debug-companion\out\extension.js:12:21370)
    at Generator.next (<anonymous>)
    at i (c:\Users\raymondzhao\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\extensions\ms-vscode.js-debug-companion\out\extension.js:1:1120)

js-debug-companion is on v1.1.1
js-debug is disabled
js-debug-nightly is on v2023.6.2717
Live Preview is on v0.5.2023062101

@luabud
Copy link
Member

luabud commented Jun 28, 2023

I also can't see the notification. Using:
js-debug-companion v1.1.1
js-debug v1.80.0
Live preview on v0.5.2023062101

Same logs as Raymond's when toggling Developer Tools.

I also set it to Firefox but I never had it installed in the first place

@andreamah
Copy link
Contributor Author

Should be fixed when microsoft/vscode-js-debug-companion#30 is adopted.

@andreamah andreamah removed the verification-found Issue verification failed label Jul 6, 2023
@mjbvz mjbvz added the verified Verification succeeded label Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants