-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[release/8.0] Fixed devtools url used for debug with chrome and edge #62080
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
base: release/8.0
Are you sure you want to change the base?
Conversation
Target net10, for now (WASM debugging is currently broken in 8, 9, pending dotnet/aspnetcore#62080
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
@lewing @wtgodbe this can go through tactics any time now and get merged on the next patch cycle. @ilonatommy validated things. |
@wtgodbe, this one also got approved. |
Hi @@github-actions[bot]. This PR was just approved to be included in the upcoming servicing release. Somebody from the @dotnet/aspnet-build team will get it merged when the branches are open. Until then, please make sure all the CI checks pass and the PR is reviewed. |
Looks like this PR hasn't been active for some time and the codebase could have been changed in the meantime. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Backport of #61948 to release/8.0
/cc @javiercn @github-actions[bot]
[release/9.0] Fixed devtools url used for debug with chrome and edge
Fixed devtools url used for debug with chrome and edge
Description
Ensures the correct creation of the devtools url with proxy to use for debug with chrome and edge browser which recently changed how urls are returned from "localhost:9222/json".
Fixes #61559
Customer Impact
Fixes a debugging issue in "debug in browser" mode.

Regression?
The format of http://localhost:9222/json changed, instead of relative uri it contains an absolute one in new browser versions.
Risk
It still supports the old way, just adds an additional fallback for cases of absolute paths.
Verification
Packaging changes reviewed?