Skip to content

Commit 250b4aa

Browse files
committed
fix(bug): delete session on 403 response from tokens updates
1 parent 765c9a6 commit 250b4aa

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

app/routes/api/auth.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -140,8 +140,6 @@ authRouter.post(
140140
'/tokens',
141141
validateRequestBody(RefreshTokenSchema),
142142
async ({ body }, res, next) => {
143-
let sessionId = ''
144-
145143
try {
146144
const { id, sid } = jwt.verify(
147145
body.refreshToken,
@@ -151,32 +149,28 @@ authRouter.post(
151149
sid: string
152150
}
153151

154-
sessionId = sid
155-
156152
const user = await prisma.user.findFirst({ where: { id } })
157153

158154
if (!user) {
159155
return next(createHttpError(403))
160156
}
161157

162158
const currentSession = await prisma.session.findFirst({
163-
where: { id: sessionId }
159+
where: { id: sid }
164160
})
165161

166162
if (!currentSession) {
167163
return next(createHttpError(403))
168164
}
169165

170-
await prisma.session.delete({ where: { id: sessionId } })
166+
await prisma.session.delete({ where: { id: sid } })
171167

172168
const newSid = await prisma.session.create({ data: { userId: user.id } })
173169

174170
const tokens = getNewTokens({ id: user.id, sid: newSid.id })
175171

176172
res.json({ ...tokens })
177173
} catch (e) {
178-
await prisma.session.delete({ where: { id: sessionId } })
179-
180174
return next(createHttpError(403))
181175
}
182176
}

0 commit comments

Comments
 (0)