Skip to content

Comments

hotfix: 학번 마스킹 처리#317

Merged
dh2906 merged 10 commits intodevelopfrom
hotfix/student-number-masking
Feb 24, 2026
Merged

hotfix: 학번 마스킹 처리#317
dh2906 merged 10 commits intodevelopfrom
hotfix/student-number-masking

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Feb 24, 2026

🔍 개요

  • close #이슈번호

🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

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

JanooGwan and others added 10 commits February 19, 2026 22:15
fix: 동아리 부원 탈퇴 관련 버그 수정
* fix: 프로메테우스 매트릭 수집 타겟을 하드코딩에서 파일 의존으로 수정

* feat: 배포 후 활성 포트 기준으로 Prometheus 타깃을 갱신하는 스크립트 추가

* fix: 모니터링 배포 경로 수정

* fix: 모니터링 배포 워크플로우 파일명 수정
@dh2906 dh2906 merged commit 73e12f5 into develop Feb 24, 2026
3 of 4 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9a38f54 and 8263cb7.

📒 Files selected for processing (1)
  • src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java

📝 Walkthrough

Walkthrough

ClubMembersResponse.java의 InnerClubMember 클래스에서 학번 필드에 대한 마스킹 기능이 추가되었습니다. 마스킹 로직은 학번의 마지막 3자리만 표시하고 나머지는 별표로 처리하며, Swagger 주석이 업데이트되었습니다.

Changes

Cohort / File(s) Summary
학번 마스킹 기능 추가
src/main/java/gg/agit/konect/domain/club/dto/ClubMembersResponse.java
InnerClubMember.studentNumber 필드에 마스킹 로직 추가. maskStudentNumber() 메서드 구현으로 null값 또는 3자리 이하일 때는 그대로 반환, 그 외에는 마지막 3자리만 표시하고 나머지는 * 처리. Swagger 주석 업데이트 및 STUDENT_NUMBER_VISIBLE_LENGTH 상수 추가.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 학번을 살짝 가려 ****123,
동아리 친구들도 안전해져요!
마지막 세 자리만 반짝 빛나고,
나머지는 비밀의 별로 변신한답니다. ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/student-number-masking

Comment @coderabbitai help to get the list of available commands and usage tips.

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