Merge pull request #10 from petschekr/master

Fix infinite redirects caused by redirecting to URL without retry token
appdevdesigns · Jun 19, 2018 · ef7d7d1 · ef7d7d1
2 parents ad0a883 + 3d02650
commit ef7d7d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/passport-cas.js b/lib/passport-cas.js
@@ -151,7 +151,7 @@ CasStrategy.prototype.authenticate = function(req, options) {
             // `ticket` portion of the querystring remains after the
             // session times out and the user refreshes the page.
             // So remove the `ticket` and try again.
-            var url = req.originalUrl || req.url
+            var url = (req.originalUrl || req.url)
                 .replace(/_cas_retry=\d+&?/, '')
                 .replace(/([?&])ticket=[\w.-]+/, '$1_cas_retry='+token);
             self.redirect(url, 307);

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "passport-cas2",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "description": "CAS 2.0 strategy for Passport.js authentication",
   "main": "./lib/index.js",
   "scripts": {