Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fixed devtools url used for debug with chrome and edge #61813

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Dona278
Copy link
Contributor

@Dona278 Dona278 commented May 6, 2025

This PR ensure 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".

Browser tested:
Edge Version 136.0.3240.50
Chrome Version 136.0.7103.49

Fixes #61559

@Dona278 Dona278 requested a review from a team as a code owner May 6, 2025 15:36
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-blazor Includes: Blazor, Razor Components label May 6, 2025
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label May 6, 2025
Copy link
Contributor

Thanks for your PR, @@Dona278. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@thaystg
Copy link
Member

thaystg commented May 8, 2025

I'm comfortable on approving the PR with the suggestions that @ilonatommy did, and also @ilonatommy if you have time can you give a double test? Maybe with an old and a new chrome/edge version?
Please let me know, if you don't have time to do it I'll try.

@javiercn
Copy link
Member

javiercn commented May 8, 2025

@thaystg I'm not an expert in this area, so if you think the change is good enough, I'm happy with it.

Do you know if this affects debugging under VS or is this just for debugging directly from the dev tools?

@Dona278
Copy link
Contributor Author

Dona278 commented May 8, 2025

If can help: with that change, VS debug still works, and because all stuffs are under "display browser debug information" context and are based only on what browser returns back if lanched with specific command, I think that VS debugging isn't affected.
Meanwhile I will update the code to reflect your the suggestions.

@Dona278
Copy link
Contributor Author

Dona278 commented May 8, 2025

@thaystg after applied suggestions I saw that the code can be changed to a simple "if it is already correct use as is, othewise keep its query string and append to the default browser devtools path". I also added comment which explain that with example of new (both chrome and edge) and old version. @javiercn now I think it is more clear the reason why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blazor WebAssembly browser debugging fails with UriFormatException using recent Chrome versions
4 participants