Skip to content

leetcode-study 외 저장소에 Week 경고 댓글이 달리는 버그 수정#25

Open
DaleSeo wants to merge 2 commits intomainfrom
fix/filter-non-allowed-repos
Open

leetcode-study 외 저장소에 Week 경고 댓글이 달리는 버그 수정#25
DaleSeo wants to merge 2 commits intomainfrom
fix/filter-non-allowed-repos

Conversation

@DaleSeo
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo commented Apr 11, 2026

Closes #24

GitHub App이 leetcode-study 외 저장소에 Week 경고 댓글을 작성하는 버그를 수정합니다. projects_v2_item webhook에는 payload.repository 필드가 없어 상위 저장소 필터를 우회하고 있었고, /check-weeks 엔드포인트에는 repo_name 검증이 빠져 있었습니다. 두 핸들러 모두 ALLOWED_REPO 상수로 검증하도록 수정하고, 회귀 테스트와 테스트 자동화를 위한 CI 워크플로우를 추가했습니다.

@DaleSeo DaleSeo self-assigned this Apr 11, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 591ce76 Commit Preview URL

Branch Preview URL
Apr 11 2026, 10:04 PM

@DaleSeo DaleSeo marked this pull request as ready for review April 11, 2026 22:14
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.

달레 UI 프로젝트의 PR에 리트코드 스터디 주차 설정 관련 코멘트가 달려서 제보드립니다

1 participant