Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
20 changes: 19 additions & 1 deletion src/Microsoft.Identity.Web.Sidecar/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ public static void Main(string[] args)
#pragma warning restore IL3057
#pragma warning restore IL2123
{
var builder = WebApplication.CreateSlimBuilder(args);
var builder = CreateBuilder(args);

builder.Services.ConfigureHttpJsonOptions(options =>
{
Expand Down Expand Up @@ -104,6 +104,24 @@ public static void Main(string[] args)
app.Run();
}

internal static WebApplicationBuilder CreateBuilder(string[] args)
{
var builder = WebApplication.CreateSlimBuilder(args);

if (!builder.Environment.IsDevelopment() &&
string.Equals(
builder.Configuration["ForwardedHeaders_Enabled"],
"true",
StringComparison.OrdinalIgnoreCase))
{
throw new InvalidOperationException(
"Forwarded headers cannot be enabled outside Development. " +
"Remove 'ForwardedHeaders_Enabled' or set it to 'false'.");
}

return builder;
}

private static void ConfigureAuthN(WebApplicationBuilder builder)
{
// Disable claims mapping.
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
// Copyright (c) Microsoft Corporation. All rights reserved.
// Licensed under the MIT License.

using Microsoft.Extensions.Hosting;
using Microsoft.Identity.Web.Sidecar;
using Xunit;

namespace Sidecar.Tests;

public class ForwardedHeadersConfigurationTests
{
[Theory]
[InlineData("Production", "true")]
[InlineData("Staging", "TrUe")]
public void Startup_ForwardedHeadersEnabledOutsideDevelopment_Throws(
string environmentName,
string value)
{
// Act & Assert
var exception = Assert.Throws<InvalidOperationException>(() =>
Program.CreateBuilder(CreateArguments(environmentName, value)));

Assert.Equal(
"Forwarded headers cannot be enabled outside Development. " +
"Remove 'ForwardedHeaders_Enabled' or set it to 'false'.",
exception.Message);
}

[Theory]
[InlineData(null)]
[InlineData("false")]
[InlineData("enabled")]
[InlineData(" true ")]
public void Startup_ForwardedHeadersNotActivatedOutsideDevelopment_Succeeds(string? value)
{
// Act
var builder = Program.CreateBuilder(CreateArguments("Production", value));

// Assert
Assert.Equal("Production", builder.Environment.EnvironmentName);
}

[Fact]
public void Startup_ForwardedHeadersEnabledInDevelopment_Succeeds()
{
// Act
var builder = Program.CreateBuilder(CreateArguments("Development", "true"));

// Assert
Assert.True(builder.Environment.IsDevelopment());
}

private static string[] CreateArguments(string environmentName, string? value)
{
if (value is null)
{
return ["--environment", environmentName];
}

return
[
"--environment",
environmentName,
"--ForwardedHeaders_Enabled",
value,
];
}
}
Loading