[8팀] 백엔드 코드리뷰용 PR

[minzix]

[8팀] Ku-Check 백엔드 코드리뷰

✨ 리뷰를 요청드리는 주요 부분

가장 고민되는 지점을 아래 형식으로 정리했습니다.

파일경로 : 조언이 필요한 포인트

• dev/src/main/kotlin/onku/backend/domain/attendance/controller : 기획 요구 상 Endpoint별 권한 분기 처리를 해야 하는데, 현업에서는 주로 이럴 때 어떤 구조로 처리하시는지 궁금합니다! 지금 저희 팀은 각 도메인 내에서 컨트롤러 파일을 역할(권한)에 따라 분리하는 구조를 취하는 중입니다.
• src/main/kotlin/onku/backend/domain/point/MemberPointHistory.kt : 상벌점 계산 시 기획 요구사항 상 각각 상벌점에 영향을 주는 Attendance, Kupick, … 등 원본 레코드들이 존재하고 이러한 레코드들의 수정사항들을 전부 저장하는 MemberHistory 테이블이 존재합니다. 이 때 점수 조회 시 계산은 원본 레코드들을 조합해서 저장하는 것과 MemberPointHistory에서 조회하는 것 중 어떤 것을 택하는게 좋을까요? 지금은 MemberPointHistory에서 조회하는 중입니다!
• src/main/kotlin/onku/backend/domain/attendance/finalize : 기획 요구 상 세션 시작 시간에서 20분을 초과 시 미출석자 전원이 결석 상태로 바뀌어야 한다는 점을 구현하기 위해 TaskScheduler 를 사용했는데 서버가 재시작 할 때, 그리고 세션을 새로이 등록할 때 외에 또 이벤트를 발행해야 할 부분이 있을까요?

💬 8팀이 고민하고 있는 부분

• 기한 상 구현에 집중하느라 곧 구현이 끝나는대로 전반적으로 리팩토링을 진행할 예정인데 어떤 부분들을 주로 수정하면 좋을지 궁금합니다!
• S3 버킷의 접근 제어, 정책 설정, 퍼블릭 액세스 차단 등 전반적인 보안 설정에 대해 조언해주시면 감사하겠습니다.
  현재 배포용 S3는 퍼블릭 액세스 모두 차단 그리고 해당 s3에 접근할 수 있는 iam정책을 만들어 배포용 ec2에 연동하여 해당 s3는 배포용 ec2에서만 액세스하도록 구현하였습니다. 그리고 ec2에서 presignedUrl(유효시간 10분) 을 발급해 클라이언트에게 전달해주면 클라이언트가 이 url로 s3에 파일을 올리거나 조회하도록 구현하였습니다.
• 저희가 앱을 배포하고 지속적으로 운영해나갈 계획인데 만약 운영관점에서 DB테이블 구조가 바뀌거나 수정이 되면 현업에서는 어떻게 수정하나요? 저희가 운영은 처음이라 예전에는 ddl auto create로 한번 초기화하고 수정을 했는데 운영은 그렇게 하면 안 될 것 같아서요

💌 요청드리는 리뷰 방향

• 코드 라인 단위 피드백뿐만 아니라, 구조/설계 관점에서의 개선 제안을 주시면 많은 도움이 될 것 같습니다.
• 코드를 살펴보시면서 과도하거나 부족한 추상화 여부, 성능 최적화 여지가 있는 부분 등에 대해 말씀해주시면 좋을 것 같습니다.

바쁘신 와중에 시간 내어 리뷰해주셔서 감사합니다.
편하게 피드백 부탁드립니다! 🥹