Skip to content

release/1.3.0#35

Merged
Sean-mn merged 47 commits intomainfrom
release/1.3.0
Apr 5, 2026
Merged

release/1.3.0#35
Sean-mn merged 47 commits intomainfrom
release/1.3.0

Conversation

@Sean-mn
Copy link
Copy Markdown
Contributor

@Sean-mn Sean-mn commented Apr 3, 2026

📚작업 내용

  • 로그인 엔드포인트에 Rate Limiting 적용 (분당 5회, Fixed Window)
  • 서비스·레포지토리 전 계층에 CancellationToken 전파
  • Room 엔터티 프로퍼티 접근 제한자를 private set으로 수정
  • Steam AppId를 환경변수(STEAM_APP_ID)로 외부 주입하도록 변경
  • CI/CD를 Docker Compose 기반 배포 방식으로 전환
  • Claude 작업 환경 재구성 (rules, skills, hooks 정비)
  • 초기 마이그레이션 통합 및 auth 스키마 정리

◀️참고 사항

Steam AppId는 기존 하드코딩(480)에서 환경변수로 변경되었습니다. 배포 시 STEAM_APP_ID 시크릿이 GitHub Actions에 등록되어 있어야 합니다.

✅체크리스트

[ ]안에 x를 작성하면 체크박스를 체크할 수 있습니다.

  • 현재 의도하고자 하는 기능이 정상적으로 작동하나요?
  • 변경한 기능이 다른 기능을 깨뜨리지 않나요?

추후 필요한 체크리스트는 업데이트 될 예정입니다.

Sean-mn added 30 commits April 1, 2026 19:47
fix: JoinRoom 비밀번호 검증 조건 및 응답 형식 개선
refactor: Claude 설정 구조 재편 및 API 응답 형식 통일
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors the project by introducing comprehensive Claude and Gemini rules, updating automation hooks, and standardizing the C# codebase. Key improvements include the systematic addition of CancellationToken support across services and repositories, the adoption of CommonApiResponse for consistent API responses, and the implementation of rate limiting. Infrastructure changes include new Docker deployment configurations and a consolidated initial database migration. Review feedback identifies a potential race condition in the user repository following the removal of atomic upsert logic, an inconsistency in rate-limiting error formats, and a layering violation where the service layer directly references API DTOs.

@Sean-mn Sean-mn merged commit 847ae1c into main Apr 5, 2026
4 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.

1 participant