Skip to content

HotFix: only check for refresh token#139

Merged
Ruthgyeul merged 1 commit intomasterfrom
develop
May 13, 2025
Merged

HotFix: only check for refresh token#139
Ruthgyeul merged 1 commit intomasterfrom
develop

Conversation

@Ruthgyeul
Copy link
Member

@Ruthgyeul Ruthgyeul commented May 13, 2025

#️⃣ Related Issue

#102

📝 PR Description

미들웨어, 리프레시 토큰 존재 유무만 체크로 변경

@Ruthgyeul Ruthgyeul requested a review from Copilot May 13, 2025 20:29
@Ruthgyeul Ruthgyeul self-assigned this May 13, 2025
@Ruthgyeul Ruthgyeul added 💻 MASTER Master PR 요청 HotFix 긴급히 오류를 고칠 때 labels May 13, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements a hotfix to change the middleware logic so that only the refresh token is checked for protecting certain paths.

  • Removed access token extraction and refresh API call logic.
  • Simplified the middleware to only redirect when the refresh token is missing.
Comments suppressed due to low confidence (1)

src/middleware.ts:3

  • API_BASE_URL is defined but not used in the new logic. Consider removing it if it is no longer needed to reduce confusion.
const API_BASE_URL = process.env.NEXT_PUBLIC_BASE_API_URL;

@Ruthgyeul Ruthgyeul merged commit d10c432 into master May 13, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HotFix 긴급히 오류를 고칠 때 💻 MASTER Master PR 요청

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants