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.

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

Bug fix client ip #85

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Bug fix client ip #85

merged 2 commits into from
Aug 22, 2024

Conversation

fnzksxl
Copy link
Contributor

@fnzksxl fnzksxl commented Aug 22, 2024

## #️⃣ 연관된 이슈

ex) #이슈번호

## 📝 작업 내용

ex) 소셜 로그인 연동

  • 프로덕션 환경에서 클라이언트의 실제 IP 취득
  • 팀 디테일 API에서 이미지 링크 수정

## 📢 참고사항

ex) Issue 내용과 달라진 점

  • 실제 IP를 취득하기 위해 수정한 conf 설정 변경은 서버에서만 이루어졌음.

fnzksxl added 2 commits August 22, 2024 10:44
@fnzksxl
#84 Bug: 환경 여부에 따라 IP 취득 헤더 수정
25c66ef 
Nginx conf에서 X-Real-IP로 remote_addr을 할당하였다.
@fnzksxl
#84 Bug: 팀 디테일 API 이미지 링크 수정
325ffae 
오타 수정
@fnzksxl fnzksxl added the bug 버그 픽스 label Aug 22, 2024
@fnzksxl fnzksxl changed the base branch from main to develop August 22, 2024 01:46
@fnzksxl fnzksxl merged commit 750f3be into develop Aug 22, 2024
@fnzksxl fnzksxl deleted the bug-fix_client_ip branch August 22, 2024 01:47
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
bug 버그 픽스
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fnzksxl