Skip to content

[이진우] rewrite 로직 && 참고용 코드 작성 #39

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
ajantang wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

[이진우] rewrite 로직 && 참고용 코드 작성 #39

ajantang wants to merge 6 commits into from

Conversation

@ajantang
Copy link
Collaborator

@ajantang ajantang commented Oct 24, 2024 •
edited
Loading

#️⃣연관된 이슈

SSR 환경에서 실행되는 API 의 쿠키 소실 이슈

📝작업 내용

rewrite 설정 추가(next.config.mjs)
SSR 용 API 함수 추가 : 기존에 커스텀 훅으로 사용되는 로직이 있어서 별도로 생성

스크린샷 (선택)

image
image
image

💬리뷰 요구사항(선택)

저 코드를 그대로 쓸 필요는 없습니다. 다만, rewrite를 통해 API 를 사용해야 하고, 서버 사이드에서 동작하는 API중 쿠키(인증)가 필요한 API는 헤더에 쿠키를 넣어줘야 합니다.
임시로 env 파일에서 설정할 baseUrl을 axios instance 생성할 때 하드 코딩으로 넣어뒀으니, 추후 env에 적용하시고 instance 코드를 수정하시면 됩니다.
vercel 쪽 fail 나는 부분은 제가 확인이 안되서 수정을 못했습니다. merge를 하게 된다면 fail 나는 부분도 확인해주셔야 합니다.
" 와 ' 차이로 수정된 코드가 많습니다.

@ajantang
test : create test page
2c150fe
@ajantang
Merge branch 'develop' into 이진우
24516e9
@ajantang
Merge branch 'develop' into 이진우
d7d888f
@ajantang
test : set cookie in getServerSideProps
ea9866c
@ajantang
feat : add rewrite logic
7b9b204 
rewrite를 이용하여, url을 재작성하여 서버에서 같은 도메인으로 인식하게 수정
BE는 배포용 설정 유지
해당 커밋은 getServerSideProps에서 실행 && 쿠키 필요한 API 수정 샘플 코드

참고 : 꼭 이 코드 방식을 따라 만들 필요없이 참고해서 SSR에서 사용되는 API 코드를 수정하시면 됩니다
@vercel
Copy link

vercel bot commented Oct 24, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
1-favorite-photo-1-fe ❌ Failed (Inspect) Oct 24, 2024 9:01am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ajantang