20240315 기술매니저님 멘토링

jjj·2024년 3월 15일
0

메모

목록 보기
18/20

20240315 기술매니저님 멘토링

  1. HTTPS 로 바꾸는건 굳이 지금 할 필요가 없을 것 같다.
    HTTPS가 아니니까 지금은 cookie를 아예 사용 못 함 -> login 라우터에서 토큰을 coockie 에 설정하는 건 지금 HTTP 환경에서는 쿠키를 사용하지 못하니 의미없는(작동을 안하는) 코드.
    삭제하고, 리프레쉬 토큰을 json({}) 에 액세스 토큰과 같이 담아서 보내는걸로 수정해야 한다.
  • { httpOnly : true } : HTTP / HTTPS 둘 다 사용 가능. HTTPS에서 cookie까지 암호화.

  • auth미들웨어에서 헤더에서 추출한 authorization이 없는 if문 메세지에
    토큰 관련해서 메세지를 준다면 더 보기 편할 것 같다.

  • 리프레쉬 토큰 라우터에 토큰 생성시, payload 부분의 id 잘 못 돼서 수정해야 함.

  • DB 한 번 다녀 오는 것도 나쁘지 않을 것 같다.

  • 주석 수정 해야 한다.

  • Joi 사용하는 것 추천.

  • 댓글 조회하는 API 지우고, 게시글 상세 조회할 때 댓글도 같이 보이게 댓글 테이블 추가.
    굳이 이유가 있다면 댓글 조회 API를 따로 작성하지만, 그게 아닌 이상 굳이? 라는 느낌.

  • 이제 할 것 리팩토링 / HTTPS 환경 / MVC 패턴으로 변경. 만약에 시간이 남는다면, Docker 사용. 배포까지 아니고, Dockerfile. 이미지 빌드까지만!

  • 리팩토링 -> Joi/expressvalidate , 에러처리미들웨어, MVC

  • readme 추가.

  • 깃허브 충돌 나는건 깃허브 사이트에서 해결 하고 pull 하면 안 꼬일 수 있다.

  • multiVote 필드 지워도 되고 안지워도 되는데 지우는 게 더 좋을 것 같다.

  • 프리즈마 generate 할 때, 어쩔 때는 prisma DB data가 지워진다는 경고 메세지가 나오고, 어쩔 때는 안 나오는데 왜 그러는지?
    프리즈마에 씽크라는 옵션이 있다. 이 옵션을 설정하면 generate?/db push? 할 때 경고 메세지가 나올 것이다.
    그런데, 우리는 따로 설정을 안 했는데 메세지가 나오는건 왜일까?ㅎㅎ
    버그인듯!

0개의 댓글

관련 채용 정보