fixes 500 error

server/app.js

@@ -52,7 +52,12 @@ app.use('/api/image', imagesRouter);
 
 // Add Auths methods
 const session = require('express-session');
-app.use(session({secret: process.env['SESSION_Secret']}));
+app.use(session({
+  secret: process.env['SESSION_Secret'],
+  resave: false,
+  saveUninitialized: false,
+  cookie: { secure: process.env.NODE_ENV === 'production' }
+}));
 
 authManager = new AuthManager(app)
 authManager.addModule('passport-js')

server/auth/modules/passport-providers/oauth.js

@@ -1,7 +1,6 @@
 var OAuth2Strategy = require('passport-oauth2')
 
 class PassportOAuth {
-
     register(app, passport, name, provider) {
         passport.use(name, new OAuth2Strategy({
             authorizationURL: provider.authorization_url,
@@ -9,35 +8,58 @@ class PassportOAuth{
             clientID: provider.client_id,
             clientSecret: provider.client_secret,
             callbackURL: `http://localhost:4400/api/auth/gmatte/callback`,
+            passReqToCallback: true
         },
-          async function(accessToken, refreshToken, params, profile, done) {
+        async function(req, accessToken, refreshToken, params, profile, done) {
             try {
-                const req = await fetch(provider.userinfo_url,{
+                const userInfoResponse = await fetch(provider.userinfo_url, {
-                    headers:{
+                    headers: { 'Authorization': `Bearer ${accessToken}` }
-                        Authorization:`Bearer ${accessToken}`
+                });
-                    }
+                const userInfo = await userInfoResponse.json();
-                })
 
-                const data = await req.json()
+                const user = {
-                profile = data
+                    id: userInfo.sub,
-                done(null,{accessToken,refreshToken,profile});
+                    email: userInfo.email,
+                    name: userInfo.name,
+                    accessToken: accessToken,
+                    refreshToken: refreshToken,
+                    expiresIn: params.expires_in
+                };
+
+                // Store the tokens in the session
+                req.session.oauth2Tokens = {
+                    accessToken: accessToken,
+                    refreshToken: refreshToken,
+                    expiresIn: params.expires_in
+                };
+
+                return done(null, user);
             } catch (error) {
+                console.error(`Error in OAuth2 Strategy ${name} :`, error);
                 return done(error);
             }
-            }
+        }));
-        ));
 
-        app.use(`/api/auth/${name}`, passport.authenticate(name,{scope: provider.scopes.join(' ') ?? 'openid profile email'}));
+        app.get(`/api/auth/${name}`, (req, res, next) => {
-        app.use(`/api/auth/${name}/callback`,
             passport.authenticate(name, {
-                successRedirect: '/', 
+                scope: provider.scopes.join(' ') ?? 'openid profile email offline_access',
-                failureRedirect: '/login',
+                prompt: 'consent'
-                session:false
+            })(req, res, next);
-            }),
+        });
-            function(accessToken, refreshToken, params, profile, cb) {
+
-                console.log(params);
+        app.get(`/api/auth/${name}/callback`, 
+            (req, res, next) => {
+                passport.authenticate(name, { failureRedirect: '/login' })(req, res, next);
+            },
+            (req, res) => {
+                if (req.user) {
+                    res.json(req.user)
+                } else {
+                    res.status(401).json({ error: 'Authentication failed' });
+                }
             }
         );
     }
 }
+
 module.exports = PassportOAuth;