thgh / payload-plugin-oauth

Add oAuth sign in to your Payload CMS site
MIT License
89 stars 16 forks source link

Plugin doesn't run in Payload 2.0 and errors out in connect-mongo #15

Open koriolis opened 1 year ago

koriolis commented 1 year ago

Hi, I'm a bit of newbie on PayloadCMS and Authentication in general. I've installed and configured the plugin as instructed on the README but unfortunately I'm running into this error which is kind of baffling me. This is the beginning of my configuration of the plugin to start testing if it works, hence the userinfo callback is very simple just logging the token it gets and returning a dummy user (I don't know what the sub property is supposed to contain, and I'm assuming it's supposed to be either the payload user id or the oauth user.id).

Anyway this fails before even executing so I'm stuck at the moment, and in the off chance this is a known solved problem, I'm opening this issue. Will be glad to provide any additional information. Thanks!

Relevant packages versions

Setup

// ...
import { oAuthPlugin } from 'payload-plugin-oauth';
// ...
export default buildConfig({
  // ...
  plugins: [
    oAuthPlugin({
      databaseUri: process.env.MONGODB_URI,
      clientID: process.env.CLIENT_ID,
      clientSecret: process.env.CLIENT_SECRET,
      authorizationURL: process.env.AUTHORIZATION_URL,
      tokenURL: process.env.TOKEN_URL,
      callbackURL: `${process.env.SERVER_URL}/api/oauth/callback`,
      async userinfo(acessToken) {
       console.dir(accessToken);
       return {
          sub: 'my-user-id-on-payload',
          name: 'John Smith',
          email: 'user@domain.com'
       };
      }
    })
  ],
  db: mongooseAdapter({
    url: process.env.MONGODB_URI
  })
});

Error

✘ [ERROR] Could not read from file: /home/my-cms/false

    node_modules/payload-plugin-oauth/dist/index.js:8:48:
      8 │ const connect_mongo_1 = __importDefault(require("connect-mongo"));
MikaelSiidorow commented 1 year ago

The plugin does not seem to support Vite as a bundler currently, theres some direct references to Webpack: https://github.com/thgh/payload-plugin-oauth/blob/3c0ea2de059e46b08b834c8713b311637c4cb7a8/src/index.ts#L205-L220

koriolis commented 1 year ago

The plugin does not seem to support Vite as a bundler currently, theres some direct references to Webpack:

https://github.com/thgh/payload-plugin-oauth/blob/3c0ea2de059e46b08b834c8713b311637c4cb7a8/src/index.ts#L205-L220

Thanks @MikaelSiidorow. I'll see if it's possible to translate these aliases into Vite and maybe add them in the global vite configuration (no sure if it's possible though).