[Bug] `X-Forwarded-Proto` 헤더 설정 제거 - Githubissues

sparcs-kaist / taxi-infra

KAIST Taxi Party Matching Web Service

https://taxi.kaist.ac.kr

MIT License

7 stars 0 forks source link

[Bug] `X-Forwarded-Proto` 헤더 설정 제거 #48

Closed withSang closed 11 months ago

withSang commented 11 months ago

Bug description

taxi-proxy nginx 컨테이너에서 X-Forwarded-Proto 헤더를 override하기 때문에 secure cookie와 rate limit이 의도한 대로 동작하지 않습니다.

secure cookie를 사용하려면 X-Forwarded-Proto 헤더 값이 "https"여야 하지만, 이 경우 "http"로 설정됩니다.

How to replicate

로그인 시도 시 "invalid request" 오류가 반환됨
- 로그인 시도 시 express에 최종적으로 보내지는 request의 X-Forwarded-Proto 헤더가 "http"로 설정되기 때문
- 세션이 존재하지 않아 sparcs sso를 사용해 로그인할 수 없음(400 error - "SparcsssoCallbackHandler : invalid request"가 반환됨)