17th-Ranger-Battalion-ORG/milsim-site-v4
8
1
Fork 0
Code Issues 38 Pull Requests 1 Actions 3 Packages Projects 1 Releases 14 Wiki Activity

Merge pull request 'removed crash causing log line' (#152) from #151-logging-crash into main
All checks were successful
Continuous Integration / Update Development (push) Successful in 2m38s
Details

Browse Source 
Reviewed-on: #152
This commit was merged in pull request #152.
This commit is contained in:
Ajdj100
2026-01-17 00:02:25 -06:00
parent c14475258d f26a334487
commit 3ff28de269
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
api/src/routes/auth.ts
View File

@@ -151,6 +151,7 @@ router.get('/callback', (req, res, next) => {
router.get('/logout', [requireLogin], function (req, res, next) { router.get('/logout', [requireLogin], function (req, res, next) {
req.logout(function (err) { req.logout(function (err) {
if (err) { return next(err); } if (err) { return next(err); }
req.session.destroy((err) => { req.session.destroy((err) => {
@@ -168,10 +169,6 @@ router.get('/logout', [requireLogin], function (req, res, next) {
returnTo: process.env.CLIENT_URL returnTo: process.env.CLIENT_URL
}; };
logger.info('auth', `Member logged out`, {
user: req.user.id,
});
res.redirect(process.env.AUTH_END_SESSION_URI + '?' + querystring.stringify(params)); res.redirect(process.env.AUTH_END_SESSION_URI + '?' + querystring.stringify(params));
}) })
}); });