Skip to content

feat: rewrites 적용#11

Merged
LeeSangHyeok0731 merged 1 commit into
developfrom
feat/rewrites
Apr 29, 2026
Merged

feat: rewrites 적용#11
LeeSangHyeok0731 merged 1 commit into
developfrom
feat/rewrites

Conversation

@LeeSangHyeok0731

Copy link
Copy Markdown
Contributor

개요 💡

서버 베포가 안정적인 환경으로 베포되어 임시로 삭제했던 rewrites 설정을 복구했습니다.

작업내용 ⌨️

  • ssh 인증 무시 제거
  • rewrites 추가
  • axiosInstance baseURL 변경

@LeeSangHyeok0731 LeeSangHyeok0731 self-assigned this Apr 28, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the API proxy configuration by removing the insecure TLS bypass in next.config.ts and implementing a rewrite rule to route /api requests to the backend. Additionally, the axiosInstance baseURL has been updated to /api. Feedback indicates that the current rewrite destination is missing the /api prefix, which may cause routing issues, and that the axios baseURL configuration results in a redundant /api/api prefix when combined with existing API paths. Suggestions were provided to correct the rewrite destination and adjust the axios baseURL.

Comment thread next.config.ts
Comment thread src/shared/lib/axios.ts
@LeeSangHyeok0731 LeeSangHyeok0731 merged commit dab1a40 into develop Apr 29, 2026
1 check passed
@LeeSangHyeok0731 LeeSangHyeok0731 deleted the feat/rewrites branch April 29, 2026 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant