Bifrost/tests/Bifrost.Web.Tests/OidcIssuerTests.cs
Meta Vens 83daabe512 Bifrost as OIDC issuer for the Matrix chat (gamecult-chat cut bifrost-oidc-issuer, squash of 1ef289a..ec4acfa and abd12b3..)
One configured client (stale clients removed with their tokens and authorizations), Active membership re-checked at token and userinfo through IsStillActiveMemberAsync, PKCE required by the server option, OpenIddict.Server.AspNetCore + EntityFrameworkCore 7.7.1 with no vulnerable transitives. Soul s2, s3 and s4 hold; Low conformance gaps in gamecult-chat:follow_up:oidc-issuer-spec-conformance.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-27 04:57:16 +01:00

737 lines
35 KiB
C#

using System.Net;
using System.Net.Http.Headers;
using System.Net.Http.Json;
using System.Security.Cryptography;
using System.Text;
using System.Text.Json;
using Bifrost.Web.Domain;
using Bifrost.Web.Features.Shared;
using Bifrost.Web.Tests.Support;
using Microsoft.AspNetCore.Mvc.Testing;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Hosting;
using Microsoft.IdentityModel.JsonWebTokens;
using Microsoft.IdentityModel.Tokens;
namespace Bifrost.Web.Tests;
public sealed class OidcIssuerTests : IClassFixture<OidcWebApplicationFactory>
{
private static readonly Uri Origin = new("https://localhost");
private readonly OidcWebApplicationFactory _factory;
public OidcIssuerTests(OidcWebApplicationFactory factory) => _factory = factory;
[Fact]
public async Task Discovery_advertises_ES256_and_the_code_flow_only()
{
using var client = NewClient();
var discovery = await client.GetFromJsonAsync<JsonElement>("/.well-known/openid-configuration");
Assert.Equal(Origin.AbsoluteUri, discovery.GetProperty("issuer").GetString());
Assert.Equal("/.well-known/jwks", new Uri(discovery.GetProperty("jwks_uri").GetString()!).AbsolutePath);
Assert.Equal(["ES256"], Strings(discovery, "id_token_signing_alg_values_supported"));
Assert.Equal(["code"], Strings(discovery, "response_types_supported"));
Assert.Equal(["authorization_code"], Strings(discovery, "grant_types_supported"));
Assert.Contains("S256", Strings(discovery, "code_challenge_methods_supported"));
Assert.Equal(["client_secret_basic"], Strings(discovery, "token_endpoint_auth_methods_supported"));
Assert.Equal(["openid", "profile"], Strings(discovery, "scopes_supported").Order());
}
[Fact]
public async Task Jwks_holds_exactly_the_configured_P256_key()
{
using var client = NewClient();
var jwks = await GetJwksAsync(client);
var key = Assert.Single(jwks.GetProperty("keys").EnumerateArray());
Assert.Equal("EC", key.GetProperty("kty").GetString());
Assert.Equal("P-256", key.GetProperty("crv").GetString());
Assert.Equal("ES256", key.GetProperty("alg").GetString());
using var ec = ECDsa.Create();
ec.ImportFromPem(_factory.Kit.SigningPublicKeyPem);
var expected = ec.ExportParameters(false);
Assert.Equal(Base64UrlEncoder.Encode(expected.Q.X), key.GetProperty("x").GetString());
Assert.Equal(Base64UrlEncoder.Encode(expected.Q.Y), key.GetProperty("y").GetString());
Assert.False(key.TryGetProperty("d", out _));
}
[Fact]
public async Task An_active_member_completes_code_and_pkce_and_gets_the_right_claims()
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, "Ada Lovelace");
var accountId = _factory.AccountId(identity);
await SetLinksAsync(identity, "heimdall-canary-7", "github-canary-7");
var tokens = await CompleteFlowAsync(client);
var verified = await ValidateIdTokenAsync(tokens.IdToken, tokens.Request.Nonce);
Assert.Equal(accountId.ToString("D"), verified.Claim("sub"));
Assert.Equal(identity, verified.Claim("preferred_username"));
Assert.Equal("Ada Lovelace", verified.Claim("name"));
Assert.Equal(["ES256"], [verified.Alg]);
// Nothing about the authenticator links, e-mail or membership leaves Bifrost.
Assert.DoesNotContain("heimdall-canary-7", verified.PayloadJson);
Assert.DoesNotContain("github-canary-7", verified.PayloadJson);
Assert.DoesNotContain("email", verified.PayloadJson, StringComparison.OrdinalIgnoreCase);
Assert.DoesNotContain("urn:bifrost", verified.PayloadJson);
using var userinfo = new HttpRequestMessage(HttpMethod.Get, "/connect/userinfo");
userinfo.Headers.Authorization = new AuthenticationHeaderValue("Bearer", tokens.AccessToken);
using var userinfoResponse = await NewClient().SendAsync(userinfo);
Assert.Equal(HttpStatusCode.OK, userinfoResponse.StatusCode);
var body = await userinfoResponse.Content.ReadAsStringAsync();
var info = JsonDocument.Parse(body).RootElement;
Assert.Equal(accountId.ToString("D"), info.GetProperty("sub").GetString());
Assert.Equal(identity, info.GetProperty("preferred_username").GetString());
Assert.Equal("Ada Lovelace", info.GetProperty("name").GetString());
Assert.DoesNotContain("canary", body);
}
[Fact]
public async Task Sub_and_preferred_username_are_stable_across_display_name_changes()
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, "First Name");
var first = await ValidateIdTokenAsync((await CompleteFlowAsync(client)).IdToken, null);
await _factory.SetDisplayNameAsync(identity, "Second Name");
var second = await ValidateIdTokenAsync((await CompleteFlowAsync(client)).IdToken, null);
Assert.Equal(first.Claim("sub"), second.Claim("sub"));
Assert.Equal(first.Claim("preferred_username"), second.Claim("preferred_username"));
Assert.Equal("First Name", first.Claim("name"));
Assert.Equal("Second Name", second.Claim("name"));
}
[Theory]
[InlineData(MembershipStatus.Authenticated)]
[InlineData(MembershipStatus.PendingApproval)]
[InlineData(MembershipStatus.Suspended)]
public async Task A_member_who_is_not_active_is_refused_with_access_denied_and_no_code(MembershipStatus status)
{
var (client, _) = await SignedInAsync(status, "Not Active");
var request = NewRequest();
var final = await DriveAsync(client, request);
var redirect = AssertRedirectToClient(final, request.State);
Assert.Equal("access_denied", redirect["error"]);
Assert.Equal(Bifrost.Web.Features.Oidc.OidcEndpoints.NotActiveDescription, redirect["error_description"]);
Assert.False(redirect.ContainsKey("code"));
}
[Fact]
public async Task An_active_account_with_no_bifrost_identity_is_refused_never_given_an_empty_username()
{
using var factory = new OidcNoIdentityFactory();
using var client = factory.CreateClient(ClientOptions());
await RegisterAsync(client, $"noid-{Guid.NewGuid():N}");
var request = NewRequest();
var final = await DriveAsync(client, request);
var redirect = AssertRedirectToClient(final, request.State);
Assert.Equal("access_denied", redirect["error"]);
Assert.Equal(Bifrost.Web.Features.Oidc.OidcEndpoints.NoIdentityDescription, redirect["error_description"]);
Assert.False(redirect.ContainsKey("code"));
}
[Theory]
[InlineData("unknown-client", OidcTestKit.RedirectUri, true)]
[InlineData(OidcTestKit.ClientId, "https://evil.example.test/callback", true)]
[InlineData(OidcTestKit.ClientId, OidcTestKit.RedirectUri, false)]
public async Task Unknown_client_wrong_redirect_uri_and_missing_pkce_are_refused_before_any_sign_in_redirect(
string clientId, string redirectUri, bool pkce)
{
using var client = NewClient();
var request = NewRequest(clientId, redirectUri, pkce);
using var response = await client.GetAsync(request.Path);
var location = response.Headers.Location?.OriginalString ?? string.Empty;
Assert.DoesNotContain("/auth/sign-in", location);
Assert.DoesNotContain("request_uri=", location);
Assert.DoesNotContain("code=", location);
Assert.NotEqual(HttpStatusCode.OK, response.StatusCode);
}
[Fact]
public async Task A_mas_sized_authorize_request_survives_sign_in_and_its_handle_is_single_use()
{
var request = NewRequest(extra: "&ui_locales=en&prompt=login&claims_locales=en&max_age=3600");
Assert.True(request.Path.Length > LocalReturnPath.MaxLength, "the request must be longer than a sign-in return path");
using var client = NewClient();
var signIn = await DriveAsync(client, request);
Assert.Equal(HttpStatusCode.Redirect, signIn.StatusCode);
Assert.Equal("/auth/sign-in", signIn.Headers.Location!.IsAbsoluteUri
? signIn.Headers.Location.AbsolutePath
: signIn.Headers.Location.OriginalString.Split('?')[0]);
var resume = ReturnUrl(signIn);
Assert.True(resume.Length <= LocalReturnPath.MaxLength, "the resume path must fit the sign-in return path");
Assert.Equal(resume, LocalReturnPath.Parse(resume).Value);
var identity = await RegisterAsync(client, $"resume-{Guid.NewGuid():N}");
await _factory.SetMembershipAsync(identity, MembershipStatus.Active);
using var resumed = await client.GetAsync(resume);
var redirect = AssertRedirectToClient(resumed, request.State);
Assert.True(redirect.ContainsKey("code"), "the resumed request issues a code");
var tokens = await ExchangeAsync(client, request, redirect["code"]!);
var verified = await ValidateIdTokenAsync(tokens.IdToken, request.Nonce);
Assert.Equal(request.Nonce, verified.Claim("nonce"));
// The handle was spent by the first resume.
using var replay = await client.GetAsync(resume);
var replayRedirect = replay.Headers.Location?.OriginalString ?? string.Empty;
Assert.DoesNotContain("code=", replayRedirect);
Assert.NotEqual(HttpStatusCode.OK, replay.StatusCode);
}
[Fact]
public async Task A_parked_authorize_request_expires()
{
// Own host: moving the clock must not skew the id_token checks of the other tests.
using var own = new OidcWebApplicationFactory();
await new OidcIssuerTests(own).ParkedRequestIsRefusedAfterItsLifetimeAsync();
}
private async Task ParkedRequestIsRefusedAfterItsLifetimeAsync()
{
// Fixed probe times, not the constant: a lifetime of a day, or of a minute, must show.
var early = NewRequest();
var late = NewRequest();
using var client = NewClient();
var earlyResume = ReturnUrl(await DriveAsync(client, early));
var lateResume = ReturnUrl(await DriveAsync(client, late));
var identity = await RegisterAsync(client, $"late-{Guid.NewGuid():N}");
await _factory.SetMembershipAsync(identity, MembershipStatus.Active);
_factory.Clock.Advance(TimeSpan.FromMinutes(5));
using var withinLifetime = await client.GetAsync(earlyResume);
Assert.True(AssertRedirectToClient(withinLifetime, early.State).ContainsKey("code"), "five minutes is within the lifetime");
_factory.Clock.Advance(TimeSpan.FromMinutes(25));
using var expired = await client.GetAsync(lateResume);
var location = expired.Headers.Location?.OriginalString ?? string.Empty;
Assert.DoesNotContain("code=", location);
Assert.NotEqual(HttpStatusCode.OK, expired.StatusCode);
}
[Fact]
public async Task The_token_endpoint_accepts_client_secret_basic_only()
{
var (client, _) = await SignedInAsync(MembershipStatus.Active, "Basic Only");
var request = NewRequest();
var code = await CodeAsync(client, request);
var wrongSecret = await TokenAsync(client, request, code, secret: "wrong-secret");
Assert.NotEqual(HttpStatusCode.OK, wrongSecret.StatusCode);
var asPost = await TokenAsync(client, request, code, secretInBody: true);
Assert.NotEqual(HttpStatusCode.OK, asPost.StatusCode);
}
[Fact]
public async Task Identity_is_minted_once()
{
using var scope = _factory.Services.CreateScope();
var service = scope.ServiceProvider.GetRequiredService<Bifrost.Web.Features.Membership.BifrostIdentityService>();
var account = new UserAccount { BifrostIdentity = "Original", NormalizedBifrostIdentity = "original" };
await service.EnsureIdentityAsync(account, "someone-else", CancellationToken.None);
Assert.Equal("original", account.NormalizedBifrostIdentity);
Assert.Equal("Original", account.BifrostIdentity);
}
[Fact]
public async Task The_openid_scope_alone_releases_the_subject_and_nothing_else()
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, "Scoped Down");
var request = NewRequest(scope: "openid");
var code = await CodeAsync(client, request);
var tokens = await ExchangeAsync(client, request, code);
var verified = await ValidateIdTokenAsync(tokens.IdToken, request.Nonce);
Assert.Equal(_factory.AccountId(identity).ToString("D"), verified.Claim("sub"));
Assert.Null(verified.Claim("preferred_username"));
Assert.Null(verified.Claim("name"));
using var userinfo = new HttpRequestMessage(HttpMethod.Get, "/connect/userinfo");
userinfo.Headers.Authorization = new AuthenticationHeaderValue("Bearer", tokens.AccessToken);
using var response = await NewClient().SendAsync(userinfo);
var info = (await response.Content.ReadFromJsonAsync<JsonElement>()).EnumerateObject().Select(x => x.Name).ToArray();
Assert.Equal(["sub"], info);
}
[Fact]
public async Task A_blank_display_name_falls_back_to_the_bifrost_identity()
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, " ");
var verified = await ValidateIdTokenAsync((await CompleteFlowAsync(client)).IdToken, null);
Assert.Equal(identity, verified.Claim("name"));
}
[Fact]
public async Task Userinfo_without_a_valid_token_is_refused()
{
using var client = NewClient();
using var none = await client.GetAsync("/connect/userinfo");
Assert.Equal(HttpStatusCode.Unauthorized, none.StatusCode);
using var forged = new HttpRequestMessage(HttpMethod.Get, "/connect/userinfo");
forged.Headers.Authorization = new AuthenticationHeaderValue("Bearer", "not-a-token");
using var forgedResponse = await client.SendAsync(forged);
Assert.Equal(HttpStatusCode.Unauthorized, forgedResponse.StatusCode);
}
[Fact]
public async Task Startup_makes_the_one_client_match_configuration_again()
{
// Own host: the drift below leaves the application manager's cache stale for every other test.
using var own = new OidcWebApplicationFactory();
await new OidcIssuerTests(own).SeederRestoresTheClientAsync();
}
private async Task SeederRestoresTheClientAsync()
{
using var warm = NewClient();
await warm.GetAsync("/healthz");
using var scope = _factory.Services.CreateScope();
var manager = scope.ServiceProvider.GetRequiredService<OpenIddict.Abstractions.IOpenIddictApplicationManager>();
var application = await manager.FindByClientIdAsync(OidcTestKit.ClientId);
Assert.NotNull(application);
// Someone widened the client in the database; the next start puts configuration back.
var drifted = new OpenIddict.Abstractions.OpenIddictApplicationDescriptor
{
ClientId = OidcTestKit.ClientId,
ClientType = OpenIddict.Abstractions.OpenIddictConstants.ClientTypes.Confidential,
ClientSecret = "drifted-secret",
RedirectUris = { new Uri("https://evil.example.test/callback") }
};
await manager.UpdateAsync(application, drifted);
var seeders = _factory.Services.GetServices<IHostedService>().OfType<Bifrost.Web.Features.Oidc.OidcClientSeeder>().ToList();
var seeder = Assert.Single(seeders);
await seeder.StartAsync(CancellationToken.None);
// Read the stored row, not through the manager: the manager's cache still holds the drifted instance.
using var fresh = _factory.Services.CreateScope();
var rows = fresh.ServiceProvider.GetRequiredService<Bifrost.Web.Data.BifrostDbContext>()
.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreApplication>()
.AsNoTracking().ToList();
var stored = Assert.Single(rows);
Assert.Equal([OidcTestKit.RedirectUri], (await manager.GetRedirectUrisAsync(stored)).ToArray());
Assert.True(await manager.ValidateClientSecretAsync(stored, _factory.Kit.ClientSecret));
Assert.False(await manager.ValidateClientSecretAsync(stored, "drifted-secret"));
}
[Theory]
[InlineData(MembershipStatus.Authenticated)]
[InlineData(MembershipStatus.PendingApproval)]
[InlineData(MembershipStatus.Suspended)]
[InlineData(null)]
public async Task A_member_no_longer_active_after_authorize_gets_no_tokens(MembershipStatus? status)
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, "Soon Gone");
var request = NewRequest();
var code = await CodeAsync(client, request);
await LeaveActiveAsync(identity, status);
using var response = await TokenAsync(client, request, code);
var body = await response.Content.ReadAsStringAsync();
Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
Assert.Contains("invalid_grant", body);
Assert.DoesNotContain("id_token", body);
Assert.DoesNotContain("access_token", body);
}
[Theory]
[InlineData(MembershipStatus.Authenticated)]
[InlineData(MembershipStatus.PendingApproval)]
[InlineData(MembershipStatus.Suspended)]
[InlineData(null)]
public async Task Userinfo_refuses_an_access_token_whose_member_is_no_longer_active(MembershipStatus? status)
{
var (client, identity) = await SignedInAsync(MembershipStatus.Active, "Soon Gone");
var tokens = await CompleteFlowAsync(client);
using (var before = await UserInfoAsync(tokens.AccessToken))
{
Assert.Equal(HttpStatusCode.OK, before.StatusCode);
}
await LeaveActiveAsync(identity, status);
using var after = await UserInfoAsync(tokens.AccessToken);
Assert.Equal(HttpStatusCode.Forbidden, after.StatusCode);
Assert.DoesNotContain(identity, await after.Content.ReadAsStringAsync());
}
private Task<HttpResponseMessage> UserInfoAsync(string accessToken)
{
var request = new HttpRequestMessage(HttpMethod.Get, "/connect/userinfo");
request.Headers.Authorization = new AuthenticationHeaderValue("Bearer", accessToken);
return NewClient().SendAsync(request);
}
/// <summary>A null status deletes the account instead.</summary>
private Task LeaveActiveAsync(string identity, MembershipStatus? status) =>
status is { } value ? _factory.SetMembershipAsync(identity, value) : _factory.DeleteAccountAsync(identity);
[Fact]
public async Task Startup_removes_every_client_but_the_configured_one_with_its_tokens_and_authorizations()
{
// Own host: the stale clients are rows the other tests must not see.
using var own = new OidcCascadeFactory();
await new OidcIssuerTests(own).StaleClientsAreRemovedAsync();
}
private async Task StaleClientsAreRemovedAsync()
{
// Two stale clients, one of them differing from the configured id only in case.
var stale = new[]
{
(Id: "old-mas", Redirect: "https://old.example.test/callback"),
(Id: OidcTestKit.ClientId.ToUpperInvariant(), Redirect: "https://shouting.example.test/callback")
};
var (client, _) = await SignedInAsync(MembershipStatus.Active, "Member");
using (var scope = _factory.Services.CreateScope())
{
var manager = scope.ServiceProvider.GetRequiredService<OpenIddict.Abstractions.IOpenIddictApplicationManager>();
foreach (var (id, redirect) in stale)
{
await manager.CreateAsync(new OpenIddict.Abstractions.OpenIddictApplicationDescriptor
{
ClientId = id,
ClientSecret = "stale-secret",
ClientType = OpenIddict.Abstractions.OpenIddictConstants.ClientTypes.Confidential,
RedirectUris = { new Uri(redirect) },
Permissions =
{
OpenIddict.Abstractions.OpenIddictConstants.Permissions.Endpoints.Authorization,
OpenIddict.Abstractions.OpenIddictConstants.Permissions.Endpoints.Token,
OpenIddict.Abstractions.OpenIddictConstants.Permissions.GrantTypes.AuthorizationCode,
OpenIddict.Abstractions.OpenIddictConstants.Permissions.ResponseTypes.Code,
OpenIddict.Abstractions.OpenIddictConstants.Permissions.Scopes.Profile,
OpenIddict.Abstractions.OpenIddictConstants.Permissions.Prefixes.Scope + OpenIddict.Abstractions.OpenIddictConstants.Scopes.OpenId
}
});
}
}
// The control: before seeding every client, stale or configured, really can obtain a code and so holds
// an authorization and a token.
foreach (var (id, redirect) in stale)
{
using var issued = await DriveAsync(client, NewRequest(id, redirect));
Assert.Contains("code=", issued.Headers.Location?.OriginalString ?? string.Empty);
}
_ = await CodeAsync(client, NewRequest());
var staleCounts = Counts(stale.Select(x => x.Id));
Assert.Equal(stale.Length, staleCounts.Count);
Assert.All(staleCounts, x => Assert.True(x.Tokens > 0 && x.Authorizations > 0, "the control must leave grants to remove"));
var configuredBefore = Counts([OidcTestKit.ClientId]).Single();
Assert.True(configuredBefore.Tokens > 0 && configuredBefore.Authorizations > 0);
var seeder = Assert.Single(_factory.Services.GetServices<IHostedService>().OfType<Bifrost.Web.Features.Oidc.OidcClientSeeder>());
await seeder.StartAsync(CancellationToken.None);
foreach (var (id, redirect) in stale)
{
using var refused = await DriveAsync(client, NewRequest(id, redirect));
Assert.DoesNotContain("code=", refused.Headers.Location?.OriginalString ?? string.Empty);
Assert.NotEqual(HttpStatusCode.OK, refused.StatusCode);
}
using var fresh = _factory.Services.CreateScope();
var db = fresh.ServiceProvider.GetRequiredService<Bifrost.Web.Data.BifrostDbContext>();
var rows = db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreApplication>().AsNoTracking().ToList();
Assert.Equal([OidcTestKit.ClientId], rows.Select(x => x.ClientId).ToArray());
// Nothing issued to a removed client survives, and the configured client's grants are untouched.
var keptId = rows.Single().Id;
Assert.Empty(db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreToken>().AsNoTracking().Where(x => x.Application!.Id != keptId));
Assert.Empty(db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreAuthorization>().AsNoTracking().Where(x => x.Application!.Id != keptId));
Assert.Equal(configuredBefore, Counts([OidcTestKit.ClientId]).Single());
}
private sealed record GrantCount(int Tokens, int Authorizations);
private List<GrantCount> Counts(IEnumerable<string> clientIds)
{
using var scope = _factory.Services.CreateScope();
var db = scope.ServiceProvider.GetRequiredService<Bifrost.Web.Data.BifrostDbContext>();
var applications = db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreApplication>().AsNoTracking().ToList();
return clientIds
.Select(id => applications.Single(x => x.ClientId == id).Id)
.Select(applicationId => new GrantCount(
db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreToken>().Count(x => x.Application!.Id == applicationId),
db.Set<OpenIddict.EntityFrameworkCore.Models.OpenIddictEntityFrameworkCoreAuthorization>().Count(x => x.Application!.Id == applicationId)))
.ToList();
}
[Fact]
public async Task The_issuer_is_the_configured_public_base_url_not_the_request_origin()
{
using var factory = new OidcForeignIssuerFactory();
using var client = factory.CreateClient(ClientOptions());
var discovery = await client.GetFromJsonAsync<JsonElement>("/.well-known/openid-configuration");
Assert.Equal(OidcForeignIssuerFactory.PublicBaseUrl, discovery.GetProperty("issuer").GetString());
}
// ---- helpers ----
private static WebApplicationFactoryClientOptions ClientOptions() => new()
{
AllowAutoRedirect = false,
HandleCookies = true,
BaseAddress = Origin
};
private HttpClient NewClient() => _factory.CreateClient(ClientOptions());
private static async Task<string> RegisterAsync(HttpClient client, string identity)
{
using var response = await client.PostAsJsonAsync("/auth/bifrost/register", new { identity, displayName = identity });
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
return identity.ToLowerInvariant();
}
private async Task<(HttpClient Client, string Identity)> SignedInAsync(MembershipStatus status, string displayName)
{
var client = NewClient();
var identity = await RegisterAsync(client, $"member-{Guid.NewGuid():N}");
await _factory.SetMembershipAsync(identity, status);
await _factory.SetDisplayNameAsync(identity, displayName);
return (client, identity);
}
private async Task SetLinksAsync(string identity, string heimdallAccountId, string gitHubLogin)
{
using var scope = _factory.Services.CreateScope();
var db = scope.ServiceProvider.GetRequiredService<Bifrost.Web.Data.BifrostDbContext>();
var account = db.UserAccounts.Single(x => x.NormalizedBifrostIdentity == identity);
account.HeimdallAccountId = heimdallAccountId;
account.GitHubLogin = gitHubLogin;
await db.SaveChangesAsync();
}
private sealed record AuthRequest(string Verifier, string State, string Nonce, string Path)
{
public string Challenge => Base64UrlEncoder.Encode(SHA256.HashData(Encoding.ASCII.GetBytes(Verifier)));
}
private static AuthRequest NewRequest(
string clientId = OidcTestKit.ClientId,
string redirectUri = OidcTestKit.RedirectUri,
bool pkce = true,
string extra = "",
string scope = "openid%20profile")
{
var verifier = Base64UrlEncoder.Encode(RandomNumberGenerator.GetBytes(48));
var state = Base64UrlEncoder.Encode(RandomNumberGenerator.GetBytes(24));
var nonce = Base64UrlEncoder.Encode(RandomNumberGenerator.GetBytes(24));
var challenge = Base64UrlEncoder.Encode(SHA256.HashData(Encoding.ASCII.GetBytes(verifier)));
var path = "/connect/authorize?response_type=code" +
$"&client_id={Uri.EscapeDataString(clientId)}" +
$"&redirect_uri={Uri.EscapeDataString(redirectUri)}" +
$"&scope={scope}" +
$"&state={state}&nonce={nonce}" +
(pkce ? $"&code_challenge={challenge}&code_challenge_method=S256" : string.Empty) +
extra;
return new AuthRequest(verifier, state, nonce, path);
}
/// <summary>Starts an authorize request and follows the provider's own redirects until the first response that is not one of them.</summary>
private static async Task<HttpResponseMessage> DriveAsync(HttpClient client, AuthRequest request)
{
var response = await client.GetAsync(request.Path);
for (var hops = 0; hops < 4 && IsAuthorizeRedirect(response); hops++)
{
var next = response.Headers.Location!;
response.Dispose();
response = await client.GetAsync(next.IsAbsoluteUri ? next.PathAndQuery : next.OriginalString);
}
return response;
}
private static bool IsAuthorizeRedirect(HttpResponseMessage response) =>
response.StatusCode is HttpStatusCode.Redirect or HttpStatusCode.Found &&
response.Headers.Location is { } location &&
(location.IsAbsoluteUri ? location.AbsolutePath : location.OriginalString).StartsWith("/connect/authorize", StringComparison.Ordinal);
private static string ReturnUrl(HttpResponseMessage signInRedirect)
{
var location = signInRedirect.Headers.Location!;
var query = location.IsAbsoluteUri ? location.Query : location.OriginalString[location.OriginalString.IndexOf('?')..];
return Microsoft.AspNetCore.WebUtilities.QueryHelpers.ParseQuery(query)
.Single(x => string.Equals(x.Key, "returnUrl", StringComparison.OrdinalIgnoreCase)).Value.ToString();
}
private static Dictionary<string, string?> AssertRedirectToClient(HttpResponseMessage response, string state)
{
Assert.True(response.StatusCode is HttpStatusCode.Redirect or HttpStatusCode.Found, $"expected a redirect, got {(int)response.StatusCode}");
var location = response.Headers.Location!;
Assert.Equal(new Uri(OidcTestKit.RedirectUri).GetLeftPart(UriPartial.Path), location.GetLeftPart(UriPartial.Path));
var query = Microsoft.AspNetCore.WebUtilities.QueryHelpers.ParseQuery(location.Query);
Assert.Equal(state, query["state"].ToString());
return query.ToDictionary(x => x.Key, x => (string?)x.Value.ToString());
}
private async Task<string> CodeAsync(HttpClient signedIn, AuthRequest request)
{
var final = await DriveAsync(signedIn, request);
return AssertRedirectToClient(final, request.State)["code"]!;
}
private sealed record Tokens(AuthRequest Request, string IdToken, string AccessToken);
private async Task<Tokens> CompleteFlowAsync(HttpClient signedIn)
{
var request = NewRequest();
var code = await CodeAsync(signedIn, request);
return await ExchangeAsync(signedIn, request, code);
}
private async Task<Tokens> ExchangeAsync(HttpClient client, AuthRequest request, string code)
{
using var response = await TokenAsync(client, request, code);
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
var body = await response.Content.ReadFromJsonAsync<JsonElement>();
Assert.DoesNotContain("refresh_token", body.EnumerateObject().Select(x => x.Name));
return new Tokens(request, body.GetProperty("id_token").GetString()!, body.GetProperty("access_token").GetString()!);
}
private async Task<HttpResponseMessage> TokenAsync(
HttpClient client, AuthRequest request, string code, string? secret = null, bool secretInBody = false)
{
var form = new Dictionary<string, string>
{
["grant_type"] = "authorization_code",
["code"] = code,
["redirect_uri"] = OidcTestKit.RedirectUri,
["code_verifier"] = request.Verifier
};
var message = new HttpRequestMessage(HttpMethod.Post, "/connect/token");
var effectiveSecret = secret ?? _factory.Kit.ClientSecret;
if (secretInBody)
{
form["client_id"] = OidcTestKit.ClientId;
form["client_secret"] = effectiveSecret;
}
else
{
message.Headers.Authorization = new AuthenticationHeaderValue(
"Basic",
Convert.ToBase64String(Encoding.ASCII.GetBytes($"{OidcTestKit.ClientId}:{effectiveSecret}")));
}
message.Content = new FormUrlEncodedContent(form);
// The code is bound to the client; use a cookieless client so a session cannot influence the exchange.
using var bare = NewClient();
return await bare.SendAsync(message);
}
private sealed record Verified(string Alg, string PayloadJson, JsonElement Payload)
{
public string? Claim(string name) => Payload.TryGetProperty(name, out var value) ? value.GetString() : null;
}
private async Task<Verified> ValidateIdTokenAsync(string idToken, string? nonce)
{
using var client = NewClient();
var jwks = new JsonWebKeySet((await GetJwksAsync(client)).GetRawText());
var result = await new JsonWebTokenHandler().ValidateTokenAsync(idToken, new TokenValidationParameters
{
ValidIssuer = Origin.AbsoluteUri,
ValidAudience = OidcTestKit.ClientId,
IssuerSigningKeys = jwks.GetSigningKeys(),
ValidAlgorithms = ["ES256"]
});
Assert.True(result.IsValid, result.Exception?.Message);
var jwt = (JsonWebToken)result.SecurityToken;
var payload = Encoding.UTF8.GetString(Base64UrlEncoder.DecodeBytes(jwt.EncodedPayload));
if (nonce is not null)
{
Assert.Equal(nonce, JsonDocument.Parse(payload).RootElement.GetProperty("nonce").GetString());
}
return new Verified(jwt.Alg, payload, JsonDocument.Parse(payload).RootElement.Clone());
}
private static async Task<JsonElement> GetJwksAsync(HttpClient client)
{
var discovery = await client.GetFromJsonAsync<JsonElement>("/.well-known/openid-configuration");
var jwksUri = new Uri(discovery.GetProperty("jwks_uri").GetString()!);
return await client.GetFromJsonAsync<JsonElement>(jwksUri.PathAndQuery);
}
private static string[] Strings(JsonElement document, string property) =>
document.GetProperty(property).EnumerateArray().Select(x => x.GetString()!).ToArray();
}
public sealed class OidcSurfaceAbsentTests : IClassFixture<TestWebApplicationFactory>
{
private readonly TestWebApplicationFactory _factory;
public OidcSurfaceAbsentTests(TestWebApplicationFactory factory) => _factory = factory;
[Theory]
[InlineData("/.well-known/openid-configuration")]
[InlineData("/.well-known/jwks")]
[InlineData("/connect/authorize")]
[InlineData("/connect/token")]
[InlineData("/connect/userinfo")]
public async Task Without_the_configuration_section_no_provider_surface_exists(string path)
{
using var client = _factory.CreateClient();
using var response = await client.GetAsync(path);
Assert.Equal(HttpStatusCode.NotFound, response.StatusCode);
}
}
public sealed class OidcStartupFailureTests
{
[Theory]
[InlineData(OidcKeyFault.MissingFile)]
[InlineData(OidcKeyFault.RsaKey)]
[InlineData(OidcKeyFault.PublicKeyOnly)]
[InlineData(OidcKeyFault.NotPem)]
public void A_configured_provider_with_an_unusable_signing_key_stops_startup_naming_the_setting_not_the_path(OidcKeyFault fault)
{
using var factory = new OidcKeyFaultFactory(fault);
var failure = Assert.ThrowsAny<Exception>(() => factory.CreateClient());
var messages = Flatten(failure).ToList();
Assert.Contains(messages, m => m.Contains("Bifrost:Oidc:SigningKeyPath", StringComparison.Ordinal));
Assert.DoesNotContain(messages, m => m.Contains("canary-", StringComparison.Ordinal));
Assert.DoesNotContain(messages, m => m.Contains("BEGIN", StringComparison.Ordinal));
}
private static IEnumerable<string> Flatten(Exception error)
{
for (var current = error; current is not null; current = current.InnerException!)
{
yield return current.Message;
if (current.InnerException is null)
{
yield break;
}
}
}
}