Skip to content

develop -> main#666

Merged
JanooGwan merged 10 commits into
mainfrom
develop
Jun 5, 2026
Merged

develop -> main#666
JanooGwan merged 10 commits into
mainfrom
develop

Conversation

@JanooGwan

Copy link
Copy Markdown
Contributor

🔍 개요


🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

JanooGwan and others added 10 commits May 31, 2026 22:08
feat: 웹 동아리 목록 정렬 옵션 추가
* feat: 대학 검색 키워드 테이블 추가

* feat: 대학 검색 약칭 데이터 추가

* refactor: 대학 검색 약칭 조회 분리

* test: 대학 검색 약칭 데이터 검증 추가

* fix: 대학 검색 키워드 마이그레이션 안정성 보강

* test: 대학 검색 키워드 시드 검증 추가

* fix: 대학 검색 키워드 연관 필드 문자열 제외
* fix: 기존 대학 검색 키워드 시드 추가

* test: 대학 검색 키워드 시드 검증 강화
* fix: 웹 대학 기준 검색 키워드 보정

* fix: 웹 대학 검색 키워드 마이그레이션 순서 보정
* feat: 웹사이트 동아리 시트 가져오기 추가

* fix: 웹사이트 동아리 시트 가져오기 안정화

* test: 웹사이트 동아리 시트 커버리지 보강
* fix: Redis TTL 조회 실패 시 rate limit 응답 안정화

- 제한 초과 판단 이후 남은 시간 조회가 실패해도 servlet 예외로 전파되지 않도록 기본 window 값으로 대체한다
- 카운터 증가가 성공한 경우에는 제한 초과 상태를 유지해 Redis TTL 조회 장애가 요청 허용으로 바뀌지 않게 한다
- Redis timeout 재현 테스트를 추가해 동일한 500 전파가 재발하지 않도록 한다

* test: rate limit TTL fallback 검증 보강

- 리뷰 피드백에 따라 TTL 조회 실패 시 기본 window 값이 예외 detail에 반영되는지 확인한다
- 429 에러 코드만 검증하던 테스트를 보강해 fallback 시간이 응답 메시지에서 누락되는 회귀를 막는다
@JanooGwan JanooGwan self-assigned this Jun 5, 2026
@JanooGwan JanooGwan added the 배포 운영 서버의 배포 관련 이슈입니다. label Jun 5, 2026
@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: c0957024-5e0a-4f19-a79b-c0d5eeef160b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

Summary

  • Overall Coverage: 87.3% ✅
  • Covered Lines: 397 / 455
  • Changed Files: 21

Coverage by File

Class Coverage Lines Status
WebUniversityRepository
gg.agit.konect.domain.website.repository
0.0% 0/2
RateLimitAspect
gg.agit.konect.global.ratelimit.aspect
78.6% 44/56
AdminWebsiteClubSheetImportService
gg.agit.konect.admin.website.service
79.0% 124/157
UniversitySearchMatcher
gg.agit.konect.domain.university.service
88.3% 53/60
WebsiteQueryRepository
gg.agit.konect.domain.website.repository
96.3% 105/109
WebsiteService
gg.agit.konect.domain.website.service
100.0% 40/40
UniversitySearchKeywordType
gg.agit.konect.domain.university.enums
100.0% 3/3
ClubRegistrationRequestController
gg.agit.konect.domain.club.controller
100.0% 4/4
AdminWebsiteClubSheetImportController
gg.agit.konect.admin.website.controller
100.0% 6/6
UniversityService
gg.agit.konect.domain.university.service
100.0% 11/11
UniversitySearchKeywordReader
gg.agit.konect.domain.university.service
100.0% 7/7

📊 View Workflow Run

@JanooGwan JanooGwan merged commit edcd119 into main Jun 5, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

배포 운영 서버의 배포 관련 이슈입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants