[service][web-app] make saml cert required, because passport-saml now…

… requires cert
ngageoint · Oct 26, 2023 · d6321f8 · d6321f8
1 parent 0944f64
commit d6321f8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/service/src/authentication/saml.js b/service/src/authentication/saml.js
@@ -15,12 +15,9 @@ function configure(strategy) {
   const options = {
     path: `/auth/${strategy.name}/callback`,
     entryPoint: strategy.settings.entryPoint,
+    cert: strategy.settings.cert,
     issuer: strategy.settings.issuer
   }
-
-  if (strategy.settings.cert) {
-    options.cert = strategy.settings.cert;
-  }
   if (strategy.settings.privateCert) {
     options.privateCert = strategy.settings.privateCert;
   }

diff --git a/...admin-authentication/admin-authentication-create/admin-authentication-create.component.ts b/...admin-authentication/admin-authentication-create/admin-authentication-create.component.ts
@@ -47,7 +47,7 @@ export class AuthenticationCreateComponent implements OnInit {
       oauth: ['clientSecret', 'clientID', 'authorizationURL', 'tokenURL', 'profileURL'],
       openidconnect: ['clientSecret', 'clientID', 'issuer', 'authorizationURL', 'tokenURL', 'profileURL'],
       ldap: ['url'],
-      saml: ['entryPoint']
+      saml: ['entryPoint', 'cert']
    }
 
    constructor(