Use the spread operator to pass in any saml configuration (#22)

SU-SWS · Nov 7, 2022 · 24fe69c · 24fe69c
1 parent ea3147f
commit 24fe69c
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/src/WebLoginAuth.ts b/src/WebLoginAuth.ts
@@ -51,13 +51,7 @@ export class WebLoginAuth {
     // Configure passport for SAML
     this.saml = new SamlStrategy(
       {
-        name:  this.config.saml.name,
-        path: this.config.saml.path,
-        callbackUrl: this.config.saml.callbackUrl,
-        issuer: this.config.saml.issuer,
         logoutUrl: this.config.saml.loginPath,
-        forceAuthn: this.config.saml.forceAuthn,
-        decryptionPvk: this.config.saml.decryptionPvk,
         entryPoint: idps[this.config.saml.idp].entryPoint,
         cert: idps[this.config.saml.idp].cert,
         wantAssertionsSigned: true,
@@ -66,6 +60,7 @@ export class WebLoginAuth {
         acceptedClockSkewMs: 60000,
         skipRequestCompression: false,
         passReqToCallback: true,
+        ...this.config.saml,
       },
       (req, profile, done) => {
         const user = attrMapper(profile);
@@ -80,7 +75,7 @@ export class WebLoginAuth {
           eduPersonPrincipalName: user.eduPersonPrincipalName,
           eduPersonScopedAffiliation: user.eduPersonScopedAffiliation,
           sn: user.sn,
-        }
+        };
         done(null, account);
       }
     );
@@ -283,7 +278,7 @@ export class WebLoginAuth {
    */
   public generateServiceProviderMetadata = () => {
     return this.saml.generateServiceProviderMetadata(this.config.saml.decryptionCert, this.config.saml.cert);
-  }
+  };
 }
 
 // Singleton client for default consumption