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

support the noDebug flag #523

Closed
captainsafia opened this issue Jun 13, 2020 · 5 comments
Closed

support the noDebug flag #523

captainsafia opened this issue Jun 13, 2020 · 5 comments
Labels
feature-request Request for new features or functionality
Milestone

Comments

@captainsafia
Copy link
Member

captainsafia commented Jun 13, 2020

This issue occurs when debugging a Blazor WASM application using vscode-js-debug.

When executing Start without Debugging, the noDebug property is set in the launching configuration for the debug session.

Screen Shot 2020-06-12 at 5 21 56 PM

However, the browser launches with debugging enabled (e.g. you can set breakpoints and it makes requests to our DebugProxy.)

Note: This happens with Chrome. I'll see if Edge has the same issue.

@connor4312
Copy link
Member

It does; js-debug does not actually respect the noDebug flag yet

@connor4312 connor4312 added the feature-request Request for new features or functionality label Jun 13, 2020
@connor4312 connor4312 changed the title noDebug flag not respected when starting application support the noDebug flag Jun 13, 2020
@captainsafia
Copy link
Member Author

Got it -- thanks for the quick response.

@isidorn
Copy link

isidorn commented Jun 15, 2020

@connor4312 I see there is already an issue for noDebug support so I will not create an additional one.

@connor4312
Copy link
Member

connor4312 commented Jun 30, 2020

I'm not sure if there's a "specification" for how noDebug is supposed to work. In the commit I have just skipped enabling the debugger, runtime, and network domains on the target. This means users will still see the sessions tree on the left hand side, but won't hit breakpoints.

Actually, I will also keep the Runtime domain enabled which lets them see user scripts, console logs, and the repl.

@connor4312 connor4312 added this to the June 2020 milestone Jul 3, 2020
@weinand
Copy link

weinand commented Jul 5, 2020

@connor4312 the DAP spec is not very precise on noDebug:

If noDebug is true the launch request should launch the program without enabling debugging.

So what you are doing for js-debug makes perfect sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality
Projects
None yet
Development

No branches or pull requests

4 participants