Skip to content

fix(security): 관리자 API 인증/권한 검증 누락 수정 - #87

Closed
lnu8926-web wants to merge 1 commit into
mainfrom
fix/admin-api-auth
Closed

fix(security): 관리자 API 인증/권한 검증 누락 수정#87
lnu8926-web wants to merge 1 commit into
mainfrom
fix/admin-api-auth

Conversation

@lnu8926-web

@lnu8926-web lnu8926-web commented May 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • 인증 없이 접근 가능했던 관리자 API 4개 수정
  • checkAdminAuth 공유 유틸 신규 생성 (src/lib/auth/adminAuth.ts)
  • announcements/route.ts의 중복 checkAdminFnc 제거

보안 수정 항목

라우트 메서드 문제 수정
admin/users GET 인증 없이 전체 유저 목록 노출 checkAdminAuth 추가
admin/users/role PATCH 인증 없이 누구나 역할 변경 가능 checkAdminAuth 추가
admin/projects GET 인증 없이 전체 프로젝트 목록 노출 checkAdminAuth 추가
admin/invitations GET/POST 인증 없음 + invited_by를 body에서 수신 checkAdminAuth 추가 + 세션에서 추출

Test plan

  • 비로그인 상태에서 /api/admin/users GET → 401 응답 확인
  • 일반 유저로 /api/admin/users/role PATCH → 403 응답 확인
  • 관리자 로그인 후 각 API 정상 동작 확인
  • 공지사항 등록/수정/삭제 관리자 기능 정상 동작 확인

Closes #78

🤖 Generated with Claude Code

- checkAdminAuth 공유 유틸 추가 (src/lib/auth/adminAuth.ts)
- admin/users GET, admin/users/role PATCH, admin/projects GET에 관리자 인증 추가
- admin/invitations GET/POST에 관리자 인증 추가
- invitations POST에서 invited_by를 body 대신 세션에서 추출하도록 수정
- announcements의 중복 checkAdminFnc 제거 후 공유 유틸로 대체

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented May 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
next_taskry Error Error May 12, 2026 2:37pm

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.

[보안] 일부 API 라우트 인증/권한 검증 누락

1 participant