Merge pull request #515 from stormpath/fix-registration-preloginhandler

Fix OAuth login case when postRegistrationHandler is set but preLoginHandler is missing
stormpath · Nov 15, 2016 · 52e9b18 · 52e9b18
2 parents ec37d68 + 5d5a578
commit 52e9b18
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/lib/helpers/login-with-oauth-provider.js b/lib/helpers/login-with-oauth-provider.js
@@ -52,6 +52,13 @@ module.exports = function loginWithOAuthProvider(options, req, res) {
     }
 
     function continueWithHandlers(preHandler, postHandler, onCompleted) {
+      // If there's no preHandler, then provide a default one.
+      if (!preHandler) {
+        preHandler = function (options, req, res, callback) {
+          callback();
+        };
+      }
+
       preHandler(options, req, res, function (err) {
         if (err) {
           return oauth.errorResponder(req, res, err);