Skip to content

Commit

Permalink
Merge pull request #1857 from tgstation/FixUpdates [NugetDeploy]
Browse files Browse the repository at this point in the history
Fix the homepage logo not working on Windows after updating
  • Loading branch information
Cyberboss authored Aug 11, 2024
2 parents 5488203 + da4754b commit d2a294a
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions src/Tgstation.Server.Host/Controllers/RootController.cs
Original file line number Diff line number Diff line change
Expand Up @@ -152,11 +152,17 @@ public IActionResult Index()
[HttpGet("logo.svg")]
public IActionResult GetLogo()
{
var logoFileName = platformIdentifier.IsWindows // these are different because of motherfucking line endings -_-
? LogoSvgWindowsName
: LogoSvgLinuxName;
// these are different because of motherfucking line endings -_-
if (platformIdentifier.IsWindows)
{
VirtualFileResult? result = this.TryServeFile(hostEnvironment, logger, $"{LogoSvgWindowsName}.svg");
if (result != null)
return result;

// BUT THE UPDATE PACKAGES ARE BUILT ON LINUX RAAAAAGH
}

return (IActionResult?)this.TryServeFile(hostEnvironment, logger, $"{logoFileName}.svg") ?? NotFound();
return (IActionResult?)this.TryServeFile(hostEnvironment, logger, $"{LogoSvgLinuxName}.svg") ?? NotFound();
}

/// <summary>
Expand Down

0 comments on commit d2a294a

Please sign in to comment.