🐛 fix: Nginx Forwarded-IP 전달 설정 및 채팅 날짜 알림 로직 수정

[asn6878]

🔨 테스크

• close 📦 chore: Let's Encrypt SSL/TLS 인증서 발급 및 Nginx 설정 추가 #300

📋 작업 내용

Nginx 설정 파일에 IP 관련 헤더 전달 설정 추가

    location /api {
       ...
        proxy_set_header Host $host; # Host 헤더로 host 변수값 전달
        proxy_set_header X-Real-IP $remote_addr;  # X-Real-IP 헤더로 실제 IP 전달
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;  # X-Forwarded-For 헤더로 실제 IP 전달
        proxy_set_header X-Forwarded-Proto $scheme; # 클라이언트의 프로토콜 전달
    }
...

https://developer.mozilla.org/ko/docs/Web/HTTP/Headers/X-Forwarded-For

• X-Real-IP 헤더 : 클라이언트의 실제 IP 주소를 단일 값으로 전달 하는 헤더. 프록시 서버가 실제 IP를 백엔드 서버에 전달하는 용도로 주로 사용됨. (의미상으로는 바로 직전의 IP 주소 가 맞는거 같긴한데, 이건 표준은 아닌 것 같아서 일단 이렇게 설정해두었습니다. 😅)

• X-Forwarded-For 헤더 : 클라이언트의 IP 주소뿐만 아니라 요청이 거쳐온 모든 프록시 서버의 IP 주소를 목록 형식으로 전달하는 헤더.

예시
클라이언트 : 1.2.3.4, 리버스 프록시 서버 : 127.0.0.1 일때, X-Forwarded-For 헤더 값은 1.2.3.4, 127.0.0.1이 됨.

• X-Forwarded-Proto 헤더 : 클라이언트의 프로토콜을 전달하는 헤더.

서버 내 IP 획득 로직 수정

CF-Connecting-IP는 CloudFlare에서 사용하는 헤더로, 이제 사용되지 않기에 제거하였습니다.
또한, request.socket?.remoteAddress 역시 Nginx에서 소켓 연결시에도 X-Forwarded-For 헤더를 설정해두도록 하였으므로 필요없어 제거 하였습니다.

채팅 오류 개선을 위한 await 키워드 추가

기존에 날짜 알림보다 채팅이 먼저 emit 되는 문제가 있어, 해당 메시지가 해당일 최초 메시지인지 확인 후 날짜 알림을 발행하는 부분에 await을 추가하여 순서를 보장할 수 있도록 처리하였습니다.

[Jo-Minseok]

고생하셨습니다! 궁금한 사안 체크해뒀습니다! getIp 코드가 중복된 함수라 따로 뽑아도 괜찮지 않을까? 생각했는데 냅둘 곳이 없네요.. 어쩔 수 없군요 ㅠ.ㅠ common에서 proxy 디렉토리 만들고 안에 getIp 함수만 따로 빼야 하나 생각도 드네요...!

[CodeVac513]

날짜 출력이 채팅보다 느린 이유가 있었군요.
민석님께서 말씀하신 것처럼 getIp로직이 현재 logger와 feed에서 사용 중인데, 앞으로 기능 추가하다가 한 번 더 반복이 된다면 그 때 유틸 따위로 getIp를 추출해도 괜찮겠네요.
수고 많으셨습니다!