Skip to content

[21기_하니홈] 프백 합동과제 최종 미션 제출합니다. #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 96 commits into
base: hanihome
Choose a base branch
from

Conversation

leerura
Copy link

@leerura leerura commented Jun 24, 2025

No description provided.

LEE and others added 30 commits May 26, 2025 09:25
[CHORE] SecurityConfig에서 cors 허용
- 공통 응답 DTO 포맷
- restBodyAdvice 구현
- 전역 예외 핸들러 구현
- http status 코드와는 별개의 ServiceCode 구현
변경: ErroCode -> ServiceCode
응답 성공에 대한 서비스코드도 가지고 있으므로 포괄적 용어 사용
Vote엔티티 시각추가 by BaseEntity
기존: /api/v1/health
변경: /health
request, respose 세부 패키지로 DTO 이동
- Election CRUD
- Candidate CRUD
- 특정 선거의 투표 결과 조회 endpoint
- 후보자별 득표수를 구하기 에서 고민한점
 Vote테이블에서 조회할 떄의 문제:
  득표수가 0인 후보를 찾아오지 못한다는거임. 그렇다고 각 후보자별로 투표수 쿼리를 날리는건 비효율적
  => Vote테이블을 조회하되 득표수가 0인 Candidate도 같이 가져오고 싶은것이니까..
  Candidate left join vote 사용.
기존: 인증없이도 조회가능한 URI임에도 JwtAuthorizationFilter에서 액세스 토큰이 없어서 exception 발생했음
변경: 토큰 없어도 다음 필터로 넘어간다.
클라이언트가 Authorization Header를 볼 수 있도록 설정
기능 상세:
- 로그인 안한 사용자 & 투표 안한 사용자: 투표 안했음을 알려줌
- 투표한 사용자: 투표했음을 나타내고 투표한 후보자 정보를 제공한다
WithFortuna and others added 30 commits June 23, 2025 22:04
허용할 오리진 추가
- 트랜잭션 적용 되지 않았기에 어노테이션 추가
…tion

[FIX]#50 CORS 트러블 슈팅 및 선거 삭제
- 도커 사용하지 않는 리소스 삭제(이미지, 컨테이너, 네트워크)
- 명령어 실패시 워크플로우 전체 실패
…n-by-section

Revert "[FEAT]#52 선거 조회 엔드포인트 추가"
- Vote를 조회할 때 해당 인스턴스에 대한 락을 걸어 멀티쓰레드 환경에서도 정상 동작하도록 예방한다
기존: 회원 아이디 요구
변경: 시큐리티컨텍스트에서 가져옴
문제상황: 현재 prune으로 이미지 정리를 할 때 현재 컨테이너에 사용되고있는 옛날 이미지는 삭제가 안되었었음. 즉, 새로운 이미지를 받기전의 old이미지와 new 이미지가 동시에 존재하는 상황이 된거임.
따라서 이미지 중복이 발생

변경: docker-compose down을 먼저하여 컨테이너를 먼저 내리고 prune을 호출
- 투표 테이블 이름이 잘못 기재돼있음. Vote -> vote로 변경..
팝업 사이클 팀명 변경
- 현재 사용중인 이미지를 삭제한 후에 신규 이미지 pull하도록 변경
[FIX]#58 ec2 도커 이미지 정리 자동화
enum값 변경으로 db 테이블 재생성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants