[release/8.0] Fixed devtools url used for debug with chrome and edge

src/Components/WebAssembly/Server/src/TargetPickerUi.cs

@@ -10,6 +10,7 @@
 using System.Text.Json;
 using System.Text.Json.Serialization;
 using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Http.Extensions;
 using System.Dynamic;
 
 namespace Microsoft.AspNetCore.Components.WebAssembly.Server;
@@ -366,9 +367,31 @@ private string GetDevToolsUrlWithProxy(BrowserTab tabToDebug)
  {
  var underlyingV8Endpoint = new Uri(tabToDebug.WebSocketDebuggerUrl);
  var proxyEndpoint = new Uri(_debugProxyUrl);
- var devToolsUrlAbsolute = new Uri(_browserHost + tabToDebug.DevtoolsFrontendUrl);
+ var devToolsUrlAbsolute = new Uri(new Uri(_browserHost), relativeUri: NormalizeDevtoolsFrontendUrl(tabToDebug.DevtoolsFrontendUrl));
  var devToolsUrlWithProxy = $"{devToolsUrlAbsolute.Scheme}://{devToolsUrlAbsolute.Authority}{devToolsUrlAbsolute.AbsolutePath}?{underlyingV8Endpoint.Scheme}={proxyEndpoint.Authority}{underlyingV8Endpoint.PathAndQuery}";
  return devToolsUrlWithProxy;
+
+ static string NormalizeDevtoolsFrontendUrl(string devtoolsFrontendUrl)
+ {
+ // Currently frontend url can be:
+ // - absolute (since v135 of chrome and edge)
+ // chrome example: https://chrome-devtools-frontend.appspot.com/serve_rev/@031848bc6ad02b97854f3d6154d3aefd0434756a/inspector.html?ws=localhost:9222/devtools/page/719FE9D3B43570193235446E0AB36859
+ // edge example: https://aka.ms/docs-landing-page/serve_rev/@4e2c41645f24197463afa2ab6aa999352ee8255c/inspector.html?ws=localhost:9222/devtools/page/3A4D56E09776321628432588FC9299F4
+ // - relative (managed as fallback for brosers with prior version)
+ // example: /devtools/inspector.html?ws=localhost:9222/devtools/page/DAB7FB6187B554E10B0BD18821265734
+ // The absolute url can't be used as-is because is not valid for debugging and cannot be made relative because of lack "devtools" segment
+ // before "inspector.html" but we can keep the query string and append to the default "devtools/inspector.html" browser devtools page
+
+ const string DefaultBrowserDevToolsPagePath = "devtools/inspector.html";
+
+ if (devtoolsFrontendUrl.AsSpan().TrimStart('/').StartsWith(DefaultBrowserDevToolsPagePath))
+ {
+ return devtoolsFrontendUrl;
+ }
+
+ UriHelper.FromAbsolute(devtoolsFrontendUrl, out _, out _, out _, out var query, out _);
+ return $"{DefaultBrowserDevToolsPagePath}{query}";
+ }
  }
 
  private string GetLaunchChromeInstructions(string targetApplicationUrl)

src/Components/test/E2ETest/ServerRenderingTests/FormHandlingTests/FormWithParentBindingContextTest.cs

@@ -1287,7 +1287,7 @@ public void CanBindToFormWithFiles()
  }
 
  [Theory]
- [InlineData(true)]
+ // [InlineData(true)] QuarantinedTest: https://github.com/dotnet/aspnetcore/issues/61882
  [InlineData(false)]
  public void CanUseFormWithMethodGet(bool suppressEnhancedNavigation)
  {