WIP: node work

inrupt · May 20, 2022 · 94daee1 · 94daee1
1 parent 0f636ba
commit 94daee1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 35 deletions.
diff --git a/...de/src/login/oidc/ClientRegistrar.spec.ts → ...login/oidc/DynamicClientRegistrar.spec.ts b/...de/src/login/oidc/ClientRegistrar.spec.ts → ...login/oidc/DynamicClientRegistrar.spec.ts
@@ -21,7 +21,7 @@
 
 import { jest, it, describe, expect } from "@jest/globals";
 import { mockStorageUtility } from "@inrupt/solid-client-authn-core";
-import ClientRegistrar from "./ClientRegistrar";
+import DynamicClientRegistrar from "./DynamicClientRegistrar";
 import {
   IssuerConfigFetcherFetchConfigResponse,
   mockDefaultIssuerMetadata,

diff --git a/...es/node/src/login/oidc/ClientRegistrar.ts → .../src/login/oidc/DynamicClientRegistrar.ts b/...es/node/src/login/oidc/ClientRegistrar.ts → .../src/login/oidc/DynamicClientRegistrar.ts
@@ -26,54 +26,25 @@
 
 import {
   IStorageUtility,
-  IClientRegistrar,
+  IDynamicClientRegistrar,
+  IDynamicClientRegistrarOptions,
   IIssuerConfig,
   IClient,
-  IClientDetails,
-  IClientRegistrarOptions,
   ConfigurationError,
-  determineSigningAlg,
   PREFERRED_SIGNING_ALG,
+  negotiateClientSigningAlg,
 } from "@inrupt/solid-client-authn-core";
 import { Client, Issuer } from "openid-client";
 import { configToIssuerMetadata } from "./IssuerConfigFetcher";
 
-export function negotiateClientSigningAlg(
-  issuerConfig: IIssuerConfig,
-  clientPreference: string[]
-): string {
-  if (!Array.isArray(issuerConfig.idTokenSigningAlgValuesSupported)) {
-    throw new Error(
-      "The OIDC issuer discovery profile is missing the 'id_token_signing_alg_values_supported' value, which is mandatory."
-    );
-  }
-
-  const signingAlg = determineSigningAlg(
-    issuerConfig.idTokenSigningAlgValuesSupported,
-    clientPreference
-  );
-
-  if (signingAlg === null) {
-    throw new Error(
-      `No signature algorithm match between ${JSON.stringify(
-        issuerConfig.idTokenSigningAlgValuesSupported
-      )} supported by the Identity Provider and ${JSON.stringify(
-        clientPreference
-      )} preferred by the client.`
-    );
-  }
-
-  return signingAlg;
-}
-
 /**
  * @hidden
  */
-export default class ClientRegistrar implements IClientRegistrar {
+export default class DynamicClientRegistrar implements IDynamicClientRegistrar {
   constructor(private storageUtility: IStorageUtility) {}
 
   async getClient(
-    options: IClientRegistrarOptions,
+    options: IDynamicClientRegistrarOptions,
     issuerConfig: IIssuerConfig
   ): Promise<IClient> {
     // If client secret and/or client id are stored in storage, use those.