fixing bug where token auth's failed to load in some cases. Closes #59

feathersjs · Aug 25, 2018 · 52b11a8 · 52b11a8
1 parent 9660492
commit 52b11a8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/packages/generator-feathers/generators/app/auth-mapping.json b/packages/generator-feathers/generators/app/auth-mapping.json
@@ -35,6 +35,7 @@
     "name": "google",
     "strategy": "passport-google-oauth20",
     "tokenStrategy": "passport-google-token",
+    "tokenStrategyExposedNormally": true,
     "permissions": {
       "scope": ["profile"]
     }
@@ -51,6 +52,7 @@
     "name": "linkedin",
     "strategy": "passport-linkedin-oauth2",
     "tokenStrategy": "passport-linkedin-token-oauth2",
+    "tokenStrategyExposedNormally": true,
     "permissions": {
       "scope": ["r_emailaddress", "r_basicprofile"]
     }

diff --git a/packages/generator-feathers/generators/app/templates/authentication.js b/packages/generator-feathers/generators/app/templates/authentication.js
@@ -3,7 +3,7 @@
 const authentication = require('feathers-authentication');
 <% for (var i = 0; i < authentication.length; i++) { %>
 const <%= S(authentication[i].name).capitalize().s %>Strategy = require('<%= authentication[i].strategy %>').Strategy;<% if (authentication[i].tokenStrategy) { %>
-const <%= S(authentication[i].name).capitalize().s %>TokenStrategy = require('<%= authentication[i].tokenStrategy %>');<% }} %>
+const <%= S(authentication[i].name).capitalize().s %>TokenStrategy = require('<%= authentication[i].tokenStrategy %>')<% if (authentication[i].tokenStrategyExposedNormally) { %>.Strategy<% } %>;<% }} %>
 
 module.exports = function() {
   const app = this;