Skip to content

[wasm][debugger] Fix justMyCode behavior #78066

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

Merged
merged 2 commits into from
Nov 29, 2022

Conversation

thaystg
Copy link
Member

@thaystg thaystg commented Nov 8, 2022

DotnetDebugger.setDebuggerProperty does not depend of having a session.

It is difficult to test things that uses sessionId as we don't have it on our debugger-tests.

Behavior without this PR: To make JustMyCode work the customer needs to turn off and turn on it after the app is already running, otherwise the default behavior is that JustMyCode is turned off.

@thaystg
Copy link
Member Author

thaystg commented Nov 11, 2022

/backport to release/7.0

@github-actions
Copy link
Contributor

Started backporting to release/7.0: https://github.com/dotnet/runtime/actions/runs/3444998775

@radical
Copy link
Member

radical commented Nov 14, 2022

Could you please describe the issue being fixed here?

@thaystg
Copy link
Member Author

thaystg commented Nov 14, 2022

There isn't any issue. Daniel Roth was testing and talked to me on teams about it.

@thaystg
Copy link
Member Author

thaystg commented Nov 18, 2022

@radical will we merge or do you want any other change?

@thaystg thaystg merged commit d4d12c5 into dotnet:main Nov 29, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Dec 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants