Skip to content

Commit 144ae66

Browse files
committed
fix: maxAge unit from ms to s
1 parent a0a811a commit 144ae66

File tree

6 files changed

+65
-24
lines changed

6 files changed

+65
-24
lines changed

packages/velog-server/src/common/constants/TimeConstants.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,16 @@ const ONE_MINUTE_IN_MS = 1000 * 60
22
const ONE_HOUR_IN_MS = ONE_MINUTE_IN_MS * 60
33
const ONE_DAY_IN_MS = ONE_HOUR_IN_MS * 24
44

5-
const ONE_MINUTE_S = 60
6-
const ONE_HOUR_S = ONE_MINUTE_S * 60
7-
const ONE_DAY_S = ONE_HOUR_S * 24
5+
const ONE_MINUTE_IN_S = 60
6+
const ONE_HOUR_IN_S = ONE_MINUTE_IN_S * 60
7+
const ONE_DAY_IN_S = ONE_HOUR_IN_S * 24
88

99
export const Time = {
1010
ONE_MINUTE_IN_MS,
1111
ONE_HOUR_IN_MS,
1212
ONE_DAY_IN_MS,
1313

14-
ONE_MINUTE_S,
15-
ONE_HOUR_S,
16-
ONE_DAY_S,
14+
ONE_MINUTE_IN_S,
15+
ONE_HOUR_IN_S,
16+
ONE_DAY_IN_S,
1717
}

packages/velog-server/src/common/plugins/global/authPlugin.ts

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,17 @@ const authPlugin: FastifyPluginAsync = async (fastify) => {
2626
if (!accessToken && !refreshToken) return
2727

2828
if (accessToken && refreshToken) {
29-
const accessTokenData = await jwt.decodeToken<AccessTokenData>(accessToken)
29+
const accessTokenData = await jwt
30+
.decodeToken<AccessTokenData>(accessToken)
31+
.catch(async () => {
32+
if (refreshToken) {
33+
const tokens = await userService.restoreToken({ request, reply })
34+
return await jwt.decodeToken<AccessTokenData>(tokens.accessToken)
35+
}
36+
})
37+
38+
if (!accessTokenData) return
39+
3040
const diff = accessTokenData.exp * 1000 - new Date().getTime()
3141
// refresh token when life < 30mins
3242
if (diff < Time.ONE_MINUTE_IN_MS * 30 && refreshToken) {

packages/velog-server/src/routes/auth/v3/social/SocialController.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,11 +130,12 @@ export class SocialController implements Controller {
130130

131131
if (user) {
132132
const tokens = await this.jwt.generateUserToken(user.id)
133+
133134
this.cookie.setCookie(reply, 'access_token', tokens.accessToken, {
134-
maxAge: Time.ONE_HOUR_IN_MS * 24,
135+
maxAge: Time.ONE_DAY_IN_S,
135136
})
136137
this.cookie.setCookie(reply, 'refresh_token', tokens.refreshToken, {
137-
maxAge: Time.ONE_DAY_IN_MS * 30,
138+
maxAge: Time.ONE_DAY_IN_S * 30,
138139
})
139140

140141
const redirectUrl = ENV.clientV3Host
@@ -255,10 +256,10 @@ export class SocialController implements Controller {
255256

256257
const tokens = await this.jwt.generateUserToken(user.id)
257258
this.cookie.setCookie(reply, 'access_token', tokens.accessToken, {
258-
maxAge: Time.ONE_HOUR_IN_MS * 24,
259+
maxAge: Time.ONE_DAY_IN_S,
259260
})
260261
this.cookie.setCookie(reply, 'refresh_token', tokens.refreshToken, {
261-
maxAge: Time.ONE_DAY_IN_MS * 30,
262+
maxAge: Time.ONE_DAY_IN_S * 30,
262263
})
263264

264265
const profile = await this.db.userProfile.findFirst({

packages/velog-server/src/services/PostService/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -463,7 +463,7 @@ export class PostService implements Service {
463463
postIds = [...postIds, ...randomPostIds]
464464
}
465465

466-
this.redis.set(cacheKey, postIds.join(','), 'EX', Time.ONE_DAY_S)
466+
this.redis.set(cacheKey, postIds.join(','), 'EX', Time.ONE_DAY_IN_S)
467467
}
468468
const posts = await this.findPostsByIds(postIds)
469469
const normalized = this.utils.normalize(posts)

packages/velog-server/src/services/UserService/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,10 +144,10 @@ export class UserService implements Service {
144144
)
145145

146146
this.cookie.setCookie(ctx.reply, 'access_token', tokens.accessToken, {
147-
maxAge: Time.ONE_HOUR_IN_MS * 24,
147+
maxAge: Time.ONE_DAY_IN_S,
148148
})
149149
this.cookie.setCookie(ctx.reply, 'refresh_token', tokens.refreshToken, {
150-
maxAge: Time.ONE_DAY_IN_MS * 30,
150+
maxAge: Time.ONE_DAY_IN_S * 30,
151151
})
152152

153153
return tokens

pnpm-lock.yaml

Lines changed: 40 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)