Move initialization of DiscordOauth2 outside of passport strategy initialization

2 years ago · cbc6e50681
parent f960edb1a1
commit cbc6e50681
1 changed files with 1 additions and 1 deletions
--- a/server/modules/authentication/discord/authentication.js
+++ b/server/modules/authentication/discord/authentication.js
@ -21,6 +21,7 @@ function hasRoles(memberRoles, authRoles) {

 module.exports = {
  init (passport, conf) {
+    const discord = new DiscordOauth2()
    passport.use(conf.key,
      new DiscordStrategy({
        clientID: conf.clientId,
@ -32,7 +33,6 @@ module.exports = {
      }, async (req, accessToken, refreshToken, profile, cb) => {
        try {
          if (conf.roles) {
-            const discord = new DiscordOauth2()
            const memberRoles = await discord.getGuildMember(accessToken, conf.guildId)
            const authRoles = conf.roles.split();
            const { roles } = await discord.getGuildMember(accessToken, conf.guildId);