Skip to content

refactor: 모든 사용자가 동아리 회비 정보에 접근할 수 있도록 변경#224

Merged
JanooGwan merged 1 commit intodevelopfrom
fix/club-apply-fee-403
Feb 14, 2026
Merged

refactor: 모든 사용자가 동아리 회비 정보에 접근할 수 있도록 변경#224
JanooGwan merged 1 commit intodevelopfrom
fix/club-apply-fee-403

Conversation

@JanooGwan
Copy link
Contributor

🔍 개요

  • close #이슈번호

🚀 주요 변경 내용

  • 기존에는 가입 완료한사람 & 관리자만 동아리 회비 정보에 접근할 수 있었는데, 관련 검증 로직을 제거하여 모든 사용자가 회비 정보에 접근할 수 있도록 변경하였습니다.

💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@JanooGwan JanooGwan requested a review from Copilot February 14, 2026 15:55
@JanooGwan JanooGwan self-assigned this Feb 14, 2026
@JanooGwan JanooGwan added the 리팩토링 리팩터링을 위한 이슈입니다. label Feb 14, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

이 PR은 동아리 회비 정보 조회 API의 접근 제어를 제거하여 모든 사용자가 회비 계좌 정보에 접근할 수 있도록 변경합니다. 기존에는 동아리에 가입 신청을 완료한 사용자 또는 동아리 관리자만 접근할 수 있었으나, 이제는 인증이 필요 없는 공개 API로 전환됩니다.

Changes:

  • 동아리 회비 정보 조회 엔드포인트에 @PublicApi 어노테이션 추가하여 인증 우회
  • getFeeInfo 메서드에서 userId 매개변수 및 관련 권한 검증 로직 제거
  • API 문서에서 권한 관련 에러 코드 제거 및 설명 간소화

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
ClubApplicationService.java getFeeInfo 메서드에서 userId 매개변수와 사용자/관리자 권한 검증 로직 제거, MANAGERS 상수 import 제거
ClubApplicationController.java getFeeInfo 메서드에서 @userid 어노테이션 및 userId 매개변수 제거
ClubApplicationApi.java @publicapi 어노테이션 추가, API 문서에서 권한 관련 설명 및 에러 코드 제거

}

public ClubFeeInfoResponse getFeeInfo(Integer clubId, Integer userId) {
public ClubFeeInfoResponse getFeeInfo(Integer clubId) {
Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 메서드는 이제 모든 사용자가 동아리 회비 계좌 정보(은행명, 계좌번호, 예금주)에 접근할 수 있도록 허용합니다. 이러한 정보가 공개되어도 보안상 문제가 없는지, 그리고 이것이 의도된 비즈니스 요구사항인지 확인이 필요합니다. 계좌 정보는 민감한 정보일 수 있으며, 동아리에 가입하지 않은 사용자에게도 노출됩니다.

Suggested change
public ClubFeeInfoResponse getFeeInfo(Integer clubId) {
public ClubFeeInfoResponse getFeeInfo(Integer clubId, Integer userId) {
userRepository.getById(userId);
clubPermissionValidator.validateManagerAccess(clubId, userId);

Copilot uses AI. Check for mistakes.
@JanooGwan JanooGwan merged commit 6eb5fdf into develop Feb 14, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

리팩토링 리팩터링을 위한 이슈입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant