Skip to content

Add separate WS and WSS ports for browser refresh server to fix TLS port conflict #52959

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.

Sign up for GitHub

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

Closed
Copilot wants to merge 4 commits into from
Closed

Add separate WS and WSS ports for browser refresh server to fix TLS port conflict #52959

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f122472
Initial plan
Copilot Feb 10, 2026
5daeaa2
Fix dotnet watch port binding when TLS is supported
Copilot Feb 10, 2026
813585f
Add separate WS and WSS ports for browser refresh server
Copilot Feb 12, 2026
e63fec3
Rename AutoReloadWebSocket* properties to BrowserWebSocket*
Copilot Feb 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion src/BuiltInTools/HotReloadClient/Web/BrowserRefreshServer.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ internal sealed class BrowserRefreshServer(
string dotnetPath,
string? autoReloadWebSocketHostName,
int? autoReloadWebSocketPort,
int? autoReloadWebSocketSecurePort,
bool suppressTimeouts)
: AbstractBrowserRefreshServer(middlewareAssemblyPath, logger, loggerFactory)
{
Expand All @@ -46,6 +47,7 @@ protected override async ValueTask<WebServerHost> CreateAndStartHostAsync(Cancel
{
var hostName = autoReloadWebSocketHostName ?? "127.0.0.1";
var port = autoReloadWebSocketPort ?? 0;
var securePort = autoReloadWebSocketSecurePort ?? 0;

var supportsTls = await IsTlsSupportedAsync(cancellationToken);

Expand All @@ -55,7 +57,7 @@ protected override async ValueTask<WebServerHost> CreateAndStartHostAsync(Cancel
builder.UseKestrel();
if (supportsTls)
{
builder.UseUrls($"https://{hostName}:{port}", $"http://{hostName}:{port}");
builder.UseUrls($"https://{hostName}:{securePort}", $"http://{hostName}:{port}");
}
else
{
Expand Down
1 change: 1 addition & 0 deletions src/BuiltInTools/Watch/AppModels/WebApplicationAppModel.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ private static string GetMiddlewareAssemblyPath()
dotnetPath: context.EnvironmentOptions.MuxerPath,
autoReloadWebSocketHostName: context.EnvironmentOptions.AutoReloadWebSocketHostName,
autoReloadWebSocketPort: context.EnvironmentOptions.AutoReloadWebSocketPort,
autoReloadWebSocketSecurePort: context.EnvironmentOptions.AutoReloadWebSocketSecurePort,
suppressTimeouts: context.EnvironmentOptions.TestFlags != TestFlags.None);
}

Expand Down
2 changes: 2 additions & 0 deletions src/BuiltInTools/Watch/Context/EnvironmentOptions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ internal sealed record EnvironmentOptions(
LogLevel? CliLogLevel = null,
string? AutoReloadWebSocketHostName = null,
int? AutoReloadWebSocketPort = null,
Comment thread
tmat marked this conversation as resolved.
Outdated
int? AutoReloadWebSocketSecurePort = null,
string? BrowserPath = null,
TestFlags TestFlags = TestFlags.None,
string TestOutput = "")
Expand All @@ -58,6 +59,7 @@ internal sealed record EnvironmentOptions(
CliLogLevel: EnvironmentVariables.CliLogLevel,
AutoReloadWebSocketHostName: EnvironmentVariables.AutoReloadWSHostName,
AutoReloadWebSocketPort: EnvironmentVariables.AutoReloadWSPort,
AutoReloadWebSocketSecurePort: EnvironmentVariables.AutoReloadWSSPort,
BrowserPath: EnvironmentVariables.BrowserPath,
TestFlags: EnvironmentVariables.TestFlags,
TestOutput: EnvironmentVariables.TestOutputDir
Expand Down
1 change: 1 addition & 0 deletions src/BuiltInTools/Watch/Context/EnvironmentVariables.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ public static LogLevel? CliLogLevel

public static string? AutoReloadWSHostName => Environment.GetEnvironmentVariable("DOTNET_WATCH_AUTO_RELOAD_WS_HOSTNAME");
public static int? AutoReloadWSPort => ReadInt("DOTNET_WATCH_AUTO_RELOAD_WS_PORT");
public static int? AutoReloadWSSPort => ReadInt("DOTNET_WATCH_AUTO_RELOAD_WSS_PORT");
public static string? BrowserPath => Environment.GetEnvironmentVariable("DOTNET_WATCH_BROWSER_PATH");

private static bool ReadBool(string variableName)
Expand Down