fix: 컨트롤러 권한(Authorization) 설정 오류 수정 (#258) #259

Merged
seonkyu.kim merged 1 commits from fix/#258-authorization-policy into develop 2026-02-28 09:22:39 +00:00
Owner

📋 작업 요약

  • Super 전용이었던 컨트롤러를 인증된 사용자 누구나 접근 가능하도록 변경
  • 인증 속성이 누락된 컨트롤러에 [Authorize] 추가

Closes #258

🛠️ 작업 내용 (Changes)

  • ServiceController.cs[Authorize(Roles = "Super")][Authorize]
  • AccountController.cs[Authorize(Roles = "Super")][Authorize]
  • MessageController.cs[Authorize] 추가 + using 추가
  • StatsController.cs[Authorize] 추가 + using 추가
  • PushController.cs[Authorize] 추가 + using 추가

📢 리뷰어 참고 사항 (To Reviewers)

  • DeviceController의 register/unregister는 외부 SDK 호출용이므로 변경하지 않음
  • Public 컨트롤러(AppVersion, Banner, Faq, Notice 등)는 의도적으로 인증 없음 유지

체크리스트 (Self Checklist)

  • 빌드(Build)가 성공적으로 수행되었는가?
  • 모든 단위 테스트(Unit Test)를 통과하였는가?
  • 불필요한 로그나 주석을 제거하였는가?
  • 컨벤션(Clean Architecture, Naming)을 준수하였는가?
  • 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가?

📸 스크린샷 / 테스트 로그 (Screenshots/Logs)

빌드 성공: 경고 0개, 오류 0개

## 📋 작업 요약 - Super 전용이었던 컨트롤러를 인증된 사용자 누구나 접근 가능하도록 변경 - 인증 속성이 누락된 컨트롤러에 `[Authorize]` 추가 ## 🔗 관련 이슈 (Related Issues) Closes #258 ## 🛠️ 작업 내용 (Changes) - [x] `ServiceController.cs` — `[Authorize(Roles = "Super")]` → `[Authorize]` - [x] `AccountController.cs` — `[Authorize(Roles = "Super")]` → `[Authorize]` - [x] `MessageController.cs` — `[Authorize]` 추가 + `using` 추가 - [x] `StatsController.cs` — `[Authorize]` 추가 + `using` 추가 - [x] `PushController.cs` — `[Authorize]` 추가 + `using` 추가 ## 📢 리뷰어 참고 사항 (To Reviewers) - DeviceController의 register/unregister는 외부 SDK 호출용이므로 변경하지 않음 - Public 컨트롤러(AppVersion, Banner, Faq, Notice 등)는 의도적으로 인증 없음 유지 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) 빌드 성공: 경고 0개, 오류 0개
seonkyu.kim added 1 commit 2026-02-28 09:16:14 +00:00
- ServiceController: [Authorize(Roles = "Super")] → [Authorize]
- AccountController: [Authorize(Roles = "Super")] → [Authorize]
- MessageController: [Authorize] 추가
- StatsController: [Authorize] 추가
- PushController: [Authorize] 추가

Closes #258
seonkyu.kim added the
Priority
High
Status
In Progress
Type
Bug
labels 2026-02-28 09:16:26 +00:00
seonkyu.kim self-assigned this 2026-02-28 09:16:29 +00:00
seonkyu.kim added this to the Phase 0: 공통 기반 + 인증 milestone 2026-02-28 09:16:35 +00:00
seonkyu.kim requested review from Owners 2026-02-28 09:16:45 +00:00
seonkyu.kim merged commit 9164d9156b into develop 2026-02-28 09:22:39 +00:00
seonkyu.kim deleted branch fix/#258-authorization-policy 2026-02-28 09:22:59 +00:00
seonkyu.kim added
Status
Done
and removed
Status
In Progress
labels 2026-02-28 09:23:53 +00:00
Sign in to join this conversation.
No description provided.