NotS1mp/Musify
0
0
Fork 0
Code Issues 23 Pull Requests Actions Packages Projects Releases Wiki Activity

Fix permanent login failure after Zitadel restarts

Browse Source 
getOidcConfig() memoized the discovery promise even when it rejected,
so a single failed OIDC discovery (e.g. Zitadel not ready yet when the
dev server starts) permanently broke /auth/login until the process was
restarted. Clear the cached promise on rejection so it retries.
This commit is contained in:
Admin
2026-09-22 16:29:58 +02:00
parent dd1cb8a5d1
commit 717e939038
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web-player/src/lib/server/auth.ts
+5 -2
View File
@@ -17,7 +17,7 @@ export interface Session extends SessionUser {
expiresAt: number; expiresAt: number;
} }
let configPromise: Promise<client.Configuration>; let configPromise: Promise<client.Configuration> | undefined;
export function getOidcConfig(): Promise<client.Configuration> { export function getOidcConfig(): Promise<client.Configuration> {
configPromise ??= (async () => { configPromise ??= (async () => {
const issuer = new URL(authConfig.issuer); const issuer = new URL(authConfig.issuer);
@@ -28,7 +28,10 @@ export function getOidcConfig(): Promise<client.Configuration> {
return secret return secret
? client.discovery(issuer, authConfig.clientId, secret, undefined, options) ? client.discovery(issuer, authConfig.clientId, secret, undefined, options)
: client.discovery(issuer, authConfig.clientId, undefined, client.None(), options); : client.discovery(issuer, authConfig.clientId, undefined, client.None(), options);
})(); })().catch((exception) => {
configPromise = undefined;
throw exception;
});
return configPromise; return configPromise;
} }