Skip to content

hotfix: skip webhook registration without admin access - #170

Merged
phnml1 merged 1 commit into
mainfrom
hotfix/169-repository-webhook-permission
May 6, 2026
Merged

hotfix: skip webhook registration without admin access#170
phnml1 merged 1 commit into
mainfrom
hotfix/169-repository-webhook-permission

Conversation

@phnml1

@phnml1 phnml1 commented May 6, 2026

Copy link
Copy Markdown
Owner

작업 유형

  • 새로운 기능 (feat)
  • 버그 수정 (fix)
  • 리팩토링 (refactor)
  • 스타일/UI (style)
  • 테스트 (test)
  • 문서 (docs)
  • 설정/환경 (chore)

관련 마일스톤

  • Week 1-2: 프로젝트 기반 & 인증
  • Week 3-4: GitHub 연동
  • Week 5-6: AI 코드 리뷰
  • Week 7-8: 실시간 협업
  • Week 9-10: 대시보드 & 배포

개요

Closes #169.

저장소 연동 시 사용자가 해당 GitHub 저장소의 관리자 권한이 없으면 webhook 생성 API가 403을 반환하던 문제를 수정합니다. GitHub 저장소 목록에서 permissions.admin 값을 받아 연결 요청에 전달하고, 서버에서는 관리자 권한이 확인된 경우에만 webhook 등록을 시도합니다.

변경 사항

  • GitHub 저장소 목록 API 응답에 canAdminister 필드를 추가했습니다.
  • 저장소 연결 요청 payload에 canAdminister를 포함했습니다.
  • POST /api/repositories에서 canAdminister === true일 때만 webhook 생성을 시도하도록 변경했습니다.
  • 저장소 목록/연동 API 테스트에 관리자 권한별 webhook 등록 동작 검증을 추가했습니다.

스크린샷 (선택)

해당 없음. 서버/API 동작 수정입니다.

테스트

  • 로컬 개발 서버에서 정상 동작 확인
  • 기존 기능에 영향 없음 확인
  • 타입 에러 없음 (tsc --noEmit)
  • ESLint 경고/에러 없음

검증 결과:

  • C:\project\CodeMate\node_modules\.bin\jest.cmd --runInBand: 통과, 22 suites / 132 tests
  • C:\project\CodeMate\node_modules\.bin\tsc.cmd --noEmit: 통과
  • C:\project\CodeMate\node_modules\.bin\eslint.cmd: 에러 0개, 기존 경고 1개 (hooks/useRealtimeComments.ts의 unused var, 본 PR 변경 범위 외)

참고 사항

관리자 권한이 없는 저장소는 webhook 기반 자동 이벤트 수신은 등록되지 않지만, 저장소 연동과 PR 동기화는 계속 진행됩니다.

@vercel

vercel Bot commented May 6, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
code-mate Ready Ready Preview, Comment May 6, 2026 7:21am

@phnml1
phnml1 merged commit a68990d into main May 6, 2026
3 checks passed
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.

hotfix: 저장소 웹훅 권한 에러 수정

1 participant