✍🏻 주요 안건

• 이동엽의 코드리뷰 global/config/common
• 동권

📑 회의 내용

코드 리뷰

1. 문자열 중복 때문에 클래스를 하나 빼는건 생각 좀 해보자
2. return 문 간단하게 okay
3. salt 값 가져오는거는 Mapper보단 BcryptEncoder에서 고고
4. 토큰을 추출하는 AuthorizationExtractor는 굳이 컴포넌트 등록 X → 내부의 메소드는 정적 메소드로 분리
5. 게시글을 조회하는 로직의 size는 내일 전체 회의에서 FE 와 상의 후 결정
6. Board 관련 테스트에서 중복되는 로직을 하나의 클래스를 생성한 뒤 안에 정적 메소드를 사용하는 방식으로 변경

이미지 관련

• 이미지 등록할 때 방식이, 이미지를 먼저 등록하고 게시글을 나중에 저장하는 방식으로 이루어진다.
  • 이렇게 되면 이미지 등록 시점에는 게시글이 없음!
  • 따라서 게시글 & 이미지 매핑에 어려움이 있음
• 이미지를 저장할 때 파일 이름을 같이 저장하는데, 이 파일 이름은 MultipartFile 타입에서만 가져올 수 있음
  • 그런데 만약 이미지를 나중에 저장하는 경우를 선택하면, 프론트에서 내려주는건 S3 링크 밖에 없어서 파일 이름은 저장 불가능

→ 민기형의 답안

• 이미지를 S3에 저장하고, 저장 시점에 게시글은 null 로 저장 ✅
  • 그 후, 게시글 저장 시점에 이미지의 게시글 id를 매핑
  • 로컬에도 UUID로 저장 ✅