Skip to content

FIX: NoticeCommandController API에 ADMIN(총동연)만 접근할 수 있도록 어노테이션 스위칭#114

Merged
hyun7586 merged 1 commit intomainfrom
fix/#113-fixing-notice-authorization
Mar 16, 2026
Merged

FIX: NoticeCommandController API에 ADMIN(총동연)만 접근할 수 있도록 어노테이션 스위칭#114
hyun7586 merged 1 commit intomainfrom
fix/#113-fixing-notice-authorization

Conversation

@hyun7586
Copy link
Contributor

@hyun7586 hyun7586 commented Mar 16, 2026

✨ 구현한 기능

  • NoticeCommandController API에 ADMIN만 접근할 수 있도록 @RequireRole 어노테이션 스위칭

📢 논의하고 싶은 내용

🎸 기타

Summary by CodeRabbit

변경 사항

  • 접근 제어
    • 공지사항 관리 기능의 필요 권한이 변경되었습니다. 이제 ADMIN 역할을 가진 사용자만 공지사항 관련 작업을 수행할 수 있습니다.

@hyun7586 hyun7586 self-assigned this Mar 16, 2026
@hyun7586 hyun7586 added the Show PR은 만들지만 머지를 바로 하는 경우 (리뷰의 여지는 남겨둠) label Mar 16, 2026
@hyun7586 hyun7586 merged commit ac1402e into main Mar 16, 2026
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 3bd8d655-b86b-46e2-ba9f-7601c2ae274d

📥 Commits

Reviewing files that changed from the base of the PR and between 0a2788c and df37de9.

📒 Files selected for processing (1)
  • src/main/java/kr/co/knuserver/presentation/notice/controller/NoticeCommandController.java

📝 Walkthrough

개요

NoticeCommandController 클래스의 접근 제어 권한 요구사항을 CLUB_ADMIN 역할에서 ADMIN 역할로 변경했습니다. 이 변경으로 해당 컨트롤러의 모든 엔드포인트에 대한 런타임 인가 정책이 수정됩니다.

변경사항

집단 / 파일 요약
접근 제어 권한 수정
src/main/java/kr/co/knuserver/presentation/notice/controller/NoticeCommandController.java
NoticeCommandController 클래스의 @RequireRole 어노테이션을 CLUB_ADMIN에서 ADMIN으로 변경하여 권한 요구사항을 상향했습니다.

예상 코드 리뷰 소요 시간

🎯 1 (Trivial) | ⏱️ ~3분

축하 시

🐰 관리자 권한으로 봄바람 불어
공지사항 컨트롤러가 더욱 견고해졌네요,
ADMIN만이 출입 가능한 궁전처럼,
접근 제어 한 줄이 보안을 지켜주니
변화는 작아도 중요한 발걸음입니다! ✨

✨ 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 fix/#113-fixing-notice-authorization
📝 Coding Plan
  • Generate coding plan for human review comments

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.

❤️ Share

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

Show PR은 만들지만 머지를 바로 하는 경우 (리뷰의 여지는 남겨둠)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant