Skip to content

[공통] 토큰 쿠키 도메인 수정#1172

Merged
Gwak-Seungju merged 3 commits intodevelopfrom
fix/#1171/fix-cookie-domain
Feb 6, 2026
Merged

[공통] 토큰 쿠키 도메인 수정#1172
Gwak-Seungju merged 3 commits intodevelopfrom
fix/#1171/fix-cookie-domain

Conversation

@Gwak-Seungju
Copy link
Contributor

What is this PR? 🔍

Changes 📝

  • 쿠키의 도메인 앞에 .을 붙였던 것을 삭제했습니다.
    • 도메인이 .koreatech.in인 쿠키가 stage.koreatech.in까지 영향을 주기 때문에 .을 삭제했습니다.

Precaution

  • 혹시 쿠키 도메인 변경으로 인해 지장이 생길 수 있다면 알려주시면 감사하겠습니다.

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@Gwak-Seungju Gwak-Seungju self-assigned this Feb 5, 2026
@Gwak-Seungju Gwak-Seungju added the 🐞 BugFix 버그 수정 label Feb 5, 2026
Copy link
Contributor

@ff1451 ff1451 left a comment

Choose a reason for hiding this comment

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

order.koreatech.in에서도 토큰 유지가 필요해서 .을 추가했었습니다
그래서 스테이지 도메인 형식을 아예 바꾸거나 스테이지와 프로덕션의 쿠키 명을 다르게 설정하는 등의 다른 방법으로 진행해야할 것 같아요..

@Gwak-Seungju
Copy link
Contributor Author

order.koreatech.in에서도 토큰 유지가 필요해서 .을 추가했었습니다 그래서 스테이지 도메인 형식을 아예 바꾸거나 스테이지와 프로덕션의 쿠키 명을 다르게 설정하는 등의 다른 방법으로 진행해야할 것 같아요..

아하 그랬었군요!! 일단 롤백하고, src/static/url.ts에서 스테이지와 프로덕션의 토큰 쿠키명을 다르게 설정했습니다.

Copy link
Contributor

@ff1451 ff1451 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@Gwak-Seungju Gwak-Seungju merged commit 669a9fd into develop Feb 6, 2026
2 checks passed
@github-actions github-actions bot deleted the fix/#1171/fix-cookie-domain branch February 6, 2026 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[공통] 인증 토큰 도메인 충돌 이슈

2 participants