From 565bf26ff8aff5be74b57757ebda345c7203e183 Mon Sep 17 00:00:00 2001 From: miniduikboot Date: Mon, 2 Jun 2025 22:52:24 +0200 Subject: [PATCH] Clarify startup warning --- src/Impostor.Server/Net/MatchmakerService.cs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/Impostor.Server/Net/MatchmakerService.cs b/src/Impostor.Server/Net/MatchmakerService.cs index eb73fb1..07ea94c 100644 --- a/src/Impostor.Server/Net/MatchmakerService.cs +++ b/src/Impostor.Server/Net/MatchmakerService.cs @@ -41,6 +41,7 @@ namespace Impostor.Server.Net _serverConfig.ResolvePublicIp(), _serverConfig.PublicPort); + var runningOutsideContainer = Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINER") == null; if (_serverConfig.PublicIp == "127.0.0.1") { // NOTE: If this warning annoys you, set your PublicIp to "localhost" @@ -48,11 +49,9 @@ namespace Impostor.Server.Net _logger.LogError("To allow people on other devices to connect to your server, change this value to your Public IP address"); _logger.LogError("For more info on how to do this see https://github.com/Impostor/Impostor/blob/master/docs/Server-configuration.md"); } - - var runningOutsideContainer = Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINER") == null; - if (_httpServerConfig.ListenIp == "0.0.0.0" && runningOutsideContainer) + else if (_httpServerConfig.ListenIp == "0.0.0.0" && runningOutsideContainer) { - _logger.LogWarning("Your HTTP server is exposed to the public internet, we recommend setting up a reverse proxy and enabling HTTPS"); + _logger.LogWarning("Since Among Us 16.0.5 it is required to support HTTPS for players to connect, we recommend setting up a reverse proxy:"); _logger.LogWarning("See https://github.com/Impostor/Impostor/blob/master/docs/Http-server.md for instructions"); } } -- 2.39.5