Skip to content

Conversation

@05AM
Copy link
Contributor

@05AM 05AM commented Jun 11, 2025

💡 유형

  • 새로운 기능 추가
  • 코드 리팩토링
  • 테스트 추가, 테스트 리팩토링

💁 해결하려는 문제를 적어주세요

  • 프로젝트 정산 상세 정보를 조회하는 API를 구현했습니다.

🤔 어떤 방식으로 해결했는지 적어주세요

  • 저번 PR에 포함되어 이미 합병된 부분이지만, spring security의 @PreAuthorize("hasRole('ADMIN')")를 사용하여 프로젝트 정산 API의 경우 ADMIN 멤버 타입이 아니라면 접근을 거부하는 로직을 추가했습니다.
public Collection<? extends GrantedAuthority> getAuthorities() {
    Set<GrantedAuthority> roles = new HashSet<>();
    roles.add(new SimpleGrantedAuthority(ROLE_PREFIX + this.memberType.toString()));
    return roles;
}

Authority를 생성할 때 멤버의 ROLE을 security가 인식할 수 있도록 prefix를 붙였습니다.

🙋 중점적으로 리뷰 했으면 하는 부분이 있다면 적어주세요

  • 프로젝트 상세 정보 matching하는 코드에서 혹시 잘못된 값을 매칭하고 있다면 알려주시면 좋을 것 같습니다.

@05AM 05AM self-assigned this Jun 11, 2025
@05AM 05AM added D-4 4일 전 까지 리뷰해주세요 feat 새로운 기능 labels Jun 11, 2025
@05AM 05AM requested a review from DuFong June 11, 2025 07:06
@github-actions github-actions bot added D-3 3일 전 까지 리뷰해주세요 D-2 2일 전 까지 리뷰해주세요 D-1 1일 전 까지 리뷰해주세요 D-0 바로 리뷰가 필요해요 and removed D-4 4일 전 까지 리뷰해주세요 D-3 3일 전 까지 리뷰해주세요 D-2 2일 전 까지 리뷰해주세요 D-1 1일 전 까지 리뷰해주세요 labels Jun 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

D-0 바로 리뷰가 필요해요 feat 새로운 기능

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants