fix(authentication-oauth): session.destroy is undefined when use cook…

…ie-session package (#2100)
feathersjs · Nov 8, 2020 · 46e84b8 · 46e84b8
1 parent 09b62c0
commit 46e84b8
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/packages/authentication-oauth/src/express.ts b/packages/authentication-oauth/src/express.ts
@@ -95,9 +95,14 @@ export default (options: OauthSetupSettings) => {
           ...payload
         };
 
-        await new Promise((resolve, reject) =>
-          req.session.destroy(err => err ? reject(err) : resolve())
-        );
+        await new Promise((resolve, reject) => {
+          if (!req.session.destroy) {
+            req.session = null;
+            resolve();
+          }
+
+          req.session.destroy(err => err ? reject(err) : resolve());
+        });
 
         debug(`Calling ${authService}.create authentication with strategy ${name}`);