Merge pull request #72 from wambotron/master

Use logoutUrl for generateLogoutRequest
node-saml · Jan 23, 2015 · 225d579 · 225d579
2 parents 33df15e + 42457e2
commit 225d579
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/passport-saml/saml.js b/lib/passport-saml/saml.js
@@ -55,6 +55,11 @@ SAML.prototype.initialize = function (options) {
       options.cacheProvider = new InMemoryCacheProvider(
           {keyExpirationPeriodMs: options.requestIdExpirationPeriodMs });
   }
+
+  if (!options.logoutUrl) {
+    // Default to Entry Point
+    options.logoutUrl = options.entryPoint || '';
+  }
 
   return options;
 };
@@ -161,7 +166,7 @@ SAML.prototype.generateLogoutRequest = function (req) {
       '@ID': id,
       '@Version': '2.0',
       '@IssueInstant': instant,
-      '@Destination': this.options.entryPoint,
+      '@Destination': this.options.logoutUrl,
       'saml:Issuer' : {
         '@xmlns:saml': 'urn:oasis:names:tc:SAML:2.0:assertion',
         '#text': this.options.issuer