Skip to content
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.

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

[FIX/#159] 새로고침 오류(redirection) 해결 #160

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

minseok1015
Copy link

@minseok1015 minseok1015 commented Nov 19, 2024

새로고침시 백엔드로 redirection 요청이 잘못가던 오류 해결하기 위해서 vite.config.js (localhost의 redirection설정) /public/_redireciton(netlify의 redirection설정)파일 수정

image

기존의 redirection 설정은 다음과 같이 되어있었음
그렇게 되면 문제가 생기게 됨

새로고침을 할 경우 url로 라우팅 설정에 의해서 itpick.netlify.app/ranking이라는 요청을 보내게 됨
그렇게 될 경우 redirection 설정이 https://itpick.netlify.app/ranking이라는 것을 https://itpick.store/ranking으로 rediretion을 하게 되어서
image

다음과 같이 백엔드로 요청을 보내고 받는 response결과가 웹 브라우저 화면상에 나타나게 돼있음

우리가 원하던 것은 백엔드 요청(예시, https://itpick.netlify.app/auth/login와 같은 요청만) 백엔드 url로 redirection(https://itpick.store/auth/login)을 원했던 것임

즉 위에서 말했던 라우팅을 위한 요청 itpick.netlify.app/ranking 에 대해서는 그냥 index.html을 반환해주면 react가 화면 보여줄 것임


결론, 그래서 백엔드 요청을 보낼때만 url 앞에 /api 를 붙여주기로 한다.

image

다음 화면을 보면
proxy를 통해서 요청에 api 가 붙은 요청은
itpick.store로 redirection을 해줄 것인데, redirection을 할 때는 api를 다시 때서 백엔드 서버로 요청을 한다
예로 들자면, https://itpick.netlify.app/api/auth/login로 요청을 보낸다면 https://itpick.store/auth/login로 요청을 바꿔서 보내겠다는 의미이다.

그렇게 되면 기존의 새로고침하는 요청 https://itpick.netlify.app/ranking과 같은 요청은 redirection 문법에 해당하지 않아서 index.html로 반환될 것임

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요. ( main ❌ / develop ⭕ )
  • PR의 제목에도 관련 이슈 번호를 포함시켜 주세요.
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

  • closed #

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • 새로고침 이슈 해결
  • localhost에서 post(회원가입..등)요청이 안가던 문제 해결

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

clicelee and others added 4 commits November 17, 2024 21:15
-반응형 최대 너비 680px
-기본 너비 100%
-모든 파일 적용 완료
-화면 이외 다른 컴포넌트에는 적용 되지 않음
새로고침시 백엔드로 redirection 요청이 잘못가던 오류 해결하기 위해서 vite.config.js (localhost의 redirection설정) /public/_redireciton(netlify의 redirection설정)파일 수정
Copy link

netlify bot commented Nov 19, 2024

Deploy Preview for itpick ready!

Name Link
🔨 Latest commit 06ed670
🔍 Latest deploy log https://app.netlify.com/sites/itpick/deploys/673d4dec3be3300008047835
😎 Deploy Preview https://deploy-preview-160--itpick.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@clicelee clicelee changed the title [DESIGN/#159] 새로고침 오류(redirection) 해결 [FIX/#159] 새로고침 오류(redirection) 해결 Nov 19, 2024
clicelee and others added 4 commits November 19, 2024 23:46
새로고침시 백엔드로 redirection 요청이 잘못가던 오류 해결하기 위해서 vite.config.js (localhost의 redirection설정) /public/_redireciton(netlify의 redirection설정)파일 수정
Copy link
Member

@yskim6772 yskim6772 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ~~ 최고 !! 🤩

const response = await axios.get('/rank/main-keywords', {

});
const response = await axios.get('/api/rank/main-keywords', {});
Copy link
Member

Choose a reason for hiding this comment

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

이런 식으로 다 수정해야겠군요!

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.

3 participants