mung-mung / FE

1 stars 0 forks source link

fix: send refresh cookie only on needed #45

Closed cimkoreauniv closed 4 months ago

cimkoreauniv commented 4 months ago

기존에는 개발 편의를 위해 axios instance의 withCredentials를 true로 설정해놓아서 항상 refresh 쿠키가 전송되었으나, 로그인 관련 기능이 안정화된 지금은 필요한 경우에만 쿠키를 보내도록 해야 할 것 같습니다.

그래서 필요한 경우에만 withCredentials를 true로 수동으로 설정하면 되는데, 현재 활성화된 부분인 로그인, 로그아웃, reissue 이외에는 쿠키를 전송할 일은 아마 없을 것이라고 판단됩니다.

만약 refresh 쿠키를 전송해야 할 일이 많이 생긴다면, 리팩토링을 통해 코드를 개선하면 될 것 같습니다.