Open
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughClubService의 getClubMembers에서 요청자의 권한(관리자 여부 및 클럽 내 포지션)을 검사해 학생번호 노출 가능 여부를 결정하고, 해당 플래그를 ClubMembersResponse로 전달하도록 변경되었습니다. DTO(InnerClubMember.from)는 마스킹 여부 플래그를 받아 학생번호를 조건부로 마스킹합니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client as "Client\n(요청자)"
participant Service as "ClubService"
participant Repo as "ClubMemberRepo"
participant DTO as "ClubMembersResponse / InnerClubMember"
Client->>Service: getClubMembers(clubId, requester)
Service->>Repo: findAllByClubId(clubId)
Repo-->>Service: List<ClubMember>
Service->>Repo: findByClubIdAndUserId(clubId, requester) (권한확인)
Repo-->>Service: Optional<ClubMember> (요청자 포지션)
Service->>Service: canViewUnmaskedStudentNumber 결정 (admin || MANAGERS)
Service->>DTO: ClubMembersResponse.from(members, !canViewUnmaskedStudentNumber)
DTO->>DTO: InnerClubMember.from(member, shouldMaskStudentNumber) 각각 호출
DTO-->>Client: ClubMembersResponse (학생번호는 플래그에 따라 마스킹)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java (1)
26-27:⚠️ Potential issue | 🟡 Minor[LEVEL: low]
studentNumberAPI 설명이 실제 응답 동작과 불일치합니다.
문제는studentNumber스키마가 “마스킹된 학번”으로 고정되어 있는데, 현재 구현은 관리자/운영진 조회 시 원본 학번을 반환한다는 점입니다.
영향은 Swagger 기반 클라이언트가 응답 계약을 잘못 해석해 권한별 개인정보 처리 로직을 오구현할 수 있으며, 운영진/관리자 계정으로 조회하면 재현됩니다.
제안은 필드 설명을 “권한에 따라 마스킹될 수 있음”으로 수정하고 예시도 마스킹/비마스킹 케이스를 모두 반영하는 것입니다.Also applies to: 41-43
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java` around lines 26 - 27, Update the `@Schema` annotation for the studentNumber field in ClubMembersResponse to reflect conditional masking (e.g., change description to "권한에 따라 마스킹될 수 있음") and replace the single masked example with a value that demonstrates both cases or an explicit note (e.g., example showing masked and unmasked forms or a combined example string). Apply the same change to the other occurrence referenced (lines 41-43) so both schema entries accurately indicate that the actual response may be masked or unmasked depending on caller permissions.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/main/java/gg/agit/konect/domain/club/service/ClubService.java`:
- Around line 213-219: The code currently calls
clubMemberRepository.existsByClubIdAndUserId(...) and then
clubMemberRepository.getByClubIdAndUserId(...), causing a TOCTOU race and extra
DB roundtrip; replace both calls with a single findByClubIdAndUserId(clubId,
userId).orElseThrow(() -> CustomException.of(FORBIDDEN_CLUB_MEMBER_ACCESS)) to
obtain the ClubMember once, then compute canViewUnmaskedStudentNumber =
MANAGERS.contains(requesterClubMember.getClubPosition()); this removes the
duplicate existence check, avoids returning NOT_FOUND_CLUB_MEMBER on a race, and
reduces database calls.
---
Outside diff comments:
In `@src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java`:
- Around line 26-27: Update the `@Schema` annotation for the studentNumber field
in ClubMembersResponse to reflect conditional masking (e.g., change description
to "권한에 따라 마스킹될 수 있음") and replace the single masked example with a value that
demonstrates both cases or an explicit note (e.g., example showing masked and
unmasked forms or a combined example string). Apply the same change to the other
occurrence referenced (lines 41-43) so both schema entries accurately indicate
that the actual response may be masked or unmasked depending on caller
permissions.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 88fac209-07bd-4489-91ad-1041f176a8dd
📒 Files selected for processing (2)
src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.javasrc/main/java/gg/agit/konect/domain/club/service/ClubService.java
📜 Review details
🧰 Additional context used
📓 Path-based instructions (2)
src/main/java/**/*.java
⚙️ CodeRabbit configuration file
src/main/java/**/*.java: 아래 원칙으로 리뷰 코멘트를 작성한다.
- 코멘트는 반드시 한국어로 작성한다.
- 반드시 수정이 필요한 항목만 코멘트로 남기고, 단순 취향 차이는 지적하지 않는다.
- 각 코멘트 첫 줄에 심각도를
[LEVEL: high|medium|low]형식으로 반드시 표기한다.- 심각도 기준: high=운영 장애 가능, medium=품질 저하, low=개선 권고.
- 각 코멘트는 "문제 -> 영향 -> 제안" 순서로 3문장 이내로 간결하게 작성한다.
- 가능하면 재현 조건 및 실패 시나리오도 포함한다.
- 제안은 현재 코드베이스(Spring Boot + JPA + Flyway) 패턴과 일치해야 한다.
- 보안, 트랜잭션 경계, 예외 처리, N+1, 성능 회귀 가능성을 우선 점검한다.
- 가독성: 변수/메서드 이름이 의도를 바로 드러내는지, 중첩과 메서드 길이가 과도하지 않은지 점검한다.
- 단순화: 불필요한 추상화, 중복 로직, 과한 방어 코드가 있으면 더 단순한 대안을 제시한다.
- 확장성: 새 요구사항 추가 시 변경 범위가 최소화되는 구조인지(하드코딩 분기/값 여부 포함) 점검한다.
Files:
src/main/java/gg/agit/konect/domain/club/service/ClubService.javasrc/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
**/*
⚙️ CodeRabbit configuration file
**/*: 공통 리뷰 톤 가이드:
- 모든 코멘트는 첫 줄에
[LEVEL: ...]태그를 포함한다.- 과장된 표현 없이 사실 기반으로 작성한다.
- 한 코멘트에는 하나의 이슈만 다룬다.
- 코드 예시가 필요하면 최소 수정 예시를 제시한다.
- 가독성/단순화/확장성 이슈를 발견하면 우선순위를 높여 코멘트한다.
Files:
src/main/java/gg/agit/konect/domain/club/service/ClubService.javasrc/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
src/main/java/gg/agit/konect/domain/club/service/ClubService.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java`:
- Around line 38-47: The current InnerClubMember.from(ClubMember clubMember,
boolean shouldMaskStudentNumber) is ambiguous and error-prone; replace the
boolean flag with a clear MaskingPolicy enum (e.g., MASKED, UNMASKED) or add two
explicit factory methods like InnerClubMember.fromMasked(ClubMember) and
InnerClubMember.fromUnmasked(ClubMember), change uses of maskStudentNumber(...)
call inside those factories accordingly, and update all call sites (including
the other occurrence around the second block) to use the enum or the new named
factories so the masking intent is explicit and cannot be inverted by mistake.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: c2bc3b0b-26cb-4a58-98f6-51b8466cbd22
📒 Files selected for processing (2)
src/main/java/gg/agit/konect/domain/club/controller/ClubBasicApi.javasrc/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
📜 Review details
🧰 Additional context used
📓 Path-based instructions (2)
src/main/java/**/*.java
⚙️ CodeRabbit configuration file
src/main/java/**/*.java: 아래 원칙으로 리뷰 코멘트를 작성한다.
- 코멘트는 반드시 한국어로 작성한다.
- 반드시 수정이 필요한 항목만 코멘트로 남기고, 단순 취향 차이는 지적하지 않는다.
- 각 코멘트 첫 줄에 심각도를
[LEVEL: high|medium|low]형식으로 반드시 표기한다.- 심각도 기준: high=운영 장애 가능, medium=품질 저하, low=개선 권고.
- 각 코멘트는 "문제 -> 영향 -> 제안" 순서로 3문장 이내로 간결하게 작성한다.
- 가능하면 재현 조건 및 실패 시나리오도 포함한다.
- 제안은 현재 코드베이스(Spring Boot + JPA + Flyway) 패턴과 일치해야 한다.
- 보안, 트랜잭션 경계, 예외 처리, N+1, 성능 회귀 가능성을 우선 점검한다.
- 가독성: 변수/메서드 이름이 의도를 바로 드러내는지, 중첩과 메서드 길이가 과도하지 않은지 점검한다.
- 단순화: 불필요한 추상화, 중복 로직, 과한 방어 코드가 있으면 더 단순한 대안을 제시한다.
- 확장성: 새 요구사항 추가 시 변경 범위가 최소화되는 구조인지(하드코딩 분기/값 여부 포함) 점검한다.
Files:
src/main/java/gg/agit/konect/domain/club/controller/ClubBasicApi.javasrc/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
**/*
⚙️ CodeRabbit configuration file
**/*: 공통 리뷰 톤 가이드:
- 모든 코멘트는 첫 줄에
[LEVEL: ...]태그를 포함한다.- 과장된 표현 없이 사실 기반으로 작성한다.
- 한 코멘트에는 하나의 이슈만 다룬다.
- 코드 예시가 필요하면 최소 수정 예시를 제시한다.
- 가독성/단순화/확장성 이슈를 발견하면 우선순위를 높여 코멘트한다.
Files:
src/main/java/gg/agit/konect/domain/club/controller/ClubBasicApi.javasrc/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
Outdated
Show resolved
Hide resolved
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔍 개요
🚀 주요 변경 내용
💬 참고 사항
✅ Checklist (완료 조건)