[BUILD] 관객 및 매니저 앱 프록시 연결 #78
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
✅ Deploy Preview for hufscheer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for hufscheer-manager ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
seongminn
approved these changes
Mar 12, 2024
apps/manager/next.config.js
Outdated
| return [ | ||
| { | ||
| source: '/api/:path*/', | ||
| destination: 'https://backoffice.hufstreaming.site/:path*/', |
Member
There was a problem hiding this comment.
destination은 여전히 환경 변수로 관리하면 보안에 더 유리하지 않을까요?
Contributor
Author
There was a problem hiding this comment.
네 저도 그렇게 생각합니다만, env 변수를 사용하는 경우 매니저 앱에서 GET 요청만 보내지는 오류가 발생하여 위와 같이 작성했습니다!
apps/spectator/next.config.js
Outdated
| return [ | ||
| { | ||
| source: '/api/:path*', | ||
| destination: 'https://api.hufstreaming.site/:path*', |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🌍 이슈 번호
✅ 작업 내용
rewrites기능을 이용하여 프록시를 적용했습니다./api/:path*경로에 요청을 보내면 서버 통신이 이루어집니다.📝 참고 자료
process.env를 사용하는 경우 매니저 앱에서 프록시 연결이 정상적으로 되지 않아(POST 요청이 아니라 GET 요청만 보내는 경우 존재)next.config.js파일 내에 명시적으로 작성했습니다..env파일이 사라져도 될 거 같습니다.♾️ 기타
ECONNREFUSED로 인한 오류가 발생하고 있는데, 찾아보니 정상적이지 않은 URL로 연결을 할 때 발생한다고 하더라구요.