feat: twitch auth module

requarks · Apr 28, 2019 · 849d4f4 · 849d4f4
1 parent d80bb92
commit 849d4f4
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 11 deletions.
diff --git a/server/models/users.js b/server/models/users.js
@@ -160,7 +160,7 @@ module.exports = class User extends Model {
 
     // Find existing user
     let user = await WIKI.models.users.query().findOne({
-      providerId: profile.id,
+      providerId: _.toString(profile.id),
       providerKey
     })
 
@@ -224,7 +224,7 @@ module.exports = class User extends Model {
       // Create account
       user = await WIKI.models.users.query().insertAndFetch({
         providerKey: providerKey,
-        providerId: profile.id,
+        providerId: _.toString(profile.id),
         email: primaryEmail,
         name: displayName,
         pictureUrl: pictureUrl,

diff --git a/server/modules/authentication/twitch/authentication.js b/server/modules/authentication/twitch/authentication.js
@@ -5,21 +5,28 @@
 // ------------------------------------
 
 const TwitchStrategy = require('passport-twitch').Strategy
+const _ = require('lodash')
 
 module.exports = {
   init (passport, conf) {
     passport.use('twitch',
       new TwitchStrategy({
         clientID: conf.clientId,
         clientSecret: conf.clientSecret,
-        callbackURL: conf.callbackURL,
-        scope: 'user_read'
-      }, function (accessToken, refreshToken, profile, cb) {
-        WIKI.models.users.processProfile(profile).then((user) => {
-          return cb(null, user) || true
-        }).catch((err) => {
-          return cb(err, null) || true
-        })
+        callbackURL: conf.callbackURL
+      }, async (accessToken, refreshToken, profile, cb) => {
+        try {
+          const user = await WIKI.models.users.processProfile({
+            profile: {
+              ...profile,
+              picture: _.get(profile, '_json.logo', '')
+            },
+            providerKey: 'twitch'
+          })
+          cb(null, user)
+        } catch (err) {
+          cb(err, null)
+        }
       }
       ))
   }

diff --git a/server/modules/authentication/twitch/definition.yml b/server/modules/authentication/twitch/definition.yml
@@ -5,8 +5,10 @@ author: requarks.io
 logo: https://static.requarks.io/logo/twitch.svg
 color: indigo darken-2
 website: https://dev.twitch.tv/docs/authentication/
-isAvailable: false
+isAvailable: true
 useForm: false
+scopes:
+  - user_read
 props:
   clientId:
     type: String