Browse Source

fix(auth): handle null SAML authnContext context

Nicolas Giard 3 years ago
parent
commit
b78026e49f
1 changed files with 2 additions and 2 deletions
  1. 2 2
      server/modules/authentication/saml/authentication.js

+ 2 - 2
server/modules/authentication/saml/authentication.js

@@ -14,14 +14,14 @@ module.exports = {
       callbackUrl: conf.callbackURL,
       callbackUrl: conf.callbackURL,
       entryPoint: conf.entryPoint,
       entryPoint: conf.entryPoint,
       issuer: conf.issuer,
       issuer: conf.issuer,
-      cert: _.split(conf.cert || '', '|'),
+      cert: (conf.cert || '').split('|'),
       signatureAlgorithm: conf.signatureAlgorithm,
       signatureAlgorithm: conf.signatureAlgorithm,
       digestAlgorithm: conf.digestAlgorithm,
       digestAlgorithm: conf.digestAlgorithm,
       identifierFormat: conf.identifierFormat,
       identifierFormat: conf.identifierFormat,
       wantAssertionsSigned: conf.wantAssertionsSigned,
       wantAssertionsSigned: conf.wantAssertionsSigned,
       acceptedClockSkewMs: _.toSafeInteger(conf.acceptedClockSkewMs),
       acceptedClockSkewMs: _.toSafeInteger(conf.acceptedClockSkewMs),
       disableRequestedAuthnContext: conf.disableRequestedAuthnContext,
       disableRequestedAuthnContext: conf.disableRequestedAuthnContext,
-      authnContext: _.split(conf.authnContext, '|'),
+      authnContext: (conf.authnContext || '').split('|'),
       racComparison: conf.racComparison,
       racComparison: conf.racComparison,
       forceAuthn: conf.forceAuthn,
       forceAuthn: conf.forceAuthn,
       passive: conf.passive,
       passive: conf.passive,