본문 바로가기

Coding/내일배움캠프

[내일배움캠프] Node.js 4기 TIL | Day 34 | 24.02.05.(월)

middlewares/need-signin.middleware.js

  • 비밀 키나 중요한 설정값은 환경 변수를 통해 관리하는 것이 보안상 더 좋습니다. 'custom-secret-key' 대신 dotenv를 사용해서 환경 변수를 적용해주세요.
  • res.clearCookie("authorization")는 두 번 중복되어 호출되고 있습니다. 에러 핸들링 블록에서 한 번만 호출하도록 리팩토링 할 수 있겠네요.

models/documents.model.js와 models/users.model.js

  • 모델 파일이 비어있네요. schema.prisma에서 사용하신다면 해당 파일은 지워주시면 될 것 같습니다!

routers/documents.router.js

  • 실제 로직이 구현되어 있지 않습니다. 다음에는 꼭 실제 로직을 구현해주세요!

routers/users.router.js

  • hasedPassword  hashedPassword로 바꿔주세요.
  • 유효성 검사에 조금 더 힘을 주어서 회원 가입을 시켜주시면 좋을 것 같습니다!

prisma/schema.prisma

  • 모델 정의가 깔끔하게 잘 되어 있네요. onDelete: Cascade 옵션을 사용한 것도 좋습니다. 사용자가 삭제될 때 관련된 문서들도 함께 삭제되어야 한다면 이 옵션을 유지해야겠죠.
  • Documents 모델이 Posts로 매핑이 되어있는데 이건 확인하고 수정 부탁드립니다!