Skip to content

fix: 결정 불필요 이슈 일괄 수정 (#376 #378 #377 #383 #381 #382 #384 #356) - #386

Merged
selee1012 merged 8 commits into
devfrom
fix/quick-issues-2026-10-30
Oct 30, 2026
Merged

fix: 결정 불필요 이슈 일괄 수정 (#376 #378 #377 #383 #381 #382 #384 #356)#386
selee1012 merged 8 commits into
devfrom
fix/quick-issues-2026-10-30

Conversation

@selee1012

@selee1012 selee1012 commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

열린 이슈 중 사용자 결정 없이 바로 고칠 수 있는 것만 모았다. 커밋 = 이슈(묶음) 단위. 마이그레이션 없음.

커밋 이슈 요지
fix(auth) #376 P1 로그아웃이 세션(family) 전체를 폐기, 갱신·로그아웃을 family 잠금으로 직렬화
fix(verification) #378 최종 실패한 sync 는 빈도 제한 슬롯 반환(트랜잭션 콜백 대신 Slot.release), 데드락 재시도는 슬롯 재획득 없음
fix(challenge) #381 #382 미등록 이미지 정리: 행을 URL 인덱스로 잠그며 재확인, 등록도 같은 행 잠금 → 직렬화. 파일 먼저 지우고 성공 시에만 행 삭제(실패 시 다음 회차 재시도)
perf(recommendation) #384 세그먼트 재집계: 30일·조건을 DB 로, (생성자,템플릿) 묶음 집계 + 생성자 일괄 조회
chore(monitoring) #356 (Refs) stg biz.members 내보내기 예외 제거, business_dashboard.py prod 전용
fix(notification) #377 약관 개정 고지: userId 커서로 끝까지, 쪽마다 커밋
fix(recommendation) #383 추천 이력 수집: (확정 시각, id) 복합 커서

남은 것(사람)

손대지 않은 것(결정 필요)

#379 #380(증빙 이미지 정책) · #375 · #373 · #372 · #368 · #357 · #355 · #345

🤖 Generated with Claude Code

selee1012 and others added 8 commits October 31, 2026 00:02
@selee1012 @claude
fix(auth): 로그아웃하면 같은 세션의 갱신 경합 토큰도 함께 폐기한다
3c2480e 
회전 유예(#371)로 같은 RT 를 동시에 갱신하면 한 세션(family)에 후속 RT 가 여럿 살아 있게 되는데,
로그아웃은 받은 RT 한 개만 폐기해 다른 후속 RT 로 계속 갱신할 수 있었다.
로그아웃은 이제 family 의 살아 있는 RT 를 모두 폐기하고, 갱신·로그아웃 모두 family 활성 RT 잠금을 먼저 잡아 직렬화한다.
유예 판단의 후속 RT 조회도 잠금 읽기로 바꿔 잠금 전 스냅샷 때문에 로그아웃된 세션이 유예를 받지 않게 했다.

Fixes #376

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
fix(verification): 실패한 동기화는 빈도 제한 슬롯을 돌려준다
fcccca2 
간격 검사가 트랜잭션 밖이라 롤백 콜백이 등록되지 않아, 실패한 sync 가 슬롯을 쥔 채 남았다.
제한기는 반환 가능한 슬롯을 돌려주고, 서비스가 데드락 재시도까지 끝내 실패하면 그 슬롯을 반환한다.
재시도는 슬롯을 다시 잡지 않으므로 429 에 걸리지 않는다. 회귀 IT 는 간격을 300초로 켜고 검증한다.

Fixes #378

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
fix(challenge): 미등록 이미지 정리는 확정된 파일만 지우고 S3 실패 시 다음 회차에 다시 시도한다
e1bad5b 
정리 배치는 후보를 한 건씩 image_url 유니크 인덱스로 잠그며 미등록을 다시 확인하고,
등록 경로(생성·수정)도 같은 행을 잠가 읽어 둘을 직렬화한다.
파일을 먼저 지우고 성공한 뒤에만 소유 기록을 지우며, 저장소 삭제 실패는 예외로 올려 그 건만 롤백한다.

Fixes #381
Fixes #382

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
perf(recommendation): 세그먼트 재집계는 최근 30일 챌린지만 DB 에서 읽는다
52c4d7d 
findAll() 로 전체 챌린지를 엔티티로 읽어 앱에서 기간·삭제·템플릿·생성자를 거르던 것을
DB 쿼리로 옮기고, (생성자, 템플릿)별 개수만 받는다. 생성자는 findAllById 로 한 번에 읽는다.
개수 n 을 세그먼트마다 더해 예전처럼 챌린지마다 +1 한 결과와 같다(경계·삭제·봇방장 IT 로 확인).

Fixes #384

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
chore(monitoring): stg 회원 구성 지표 예외를 지운다
d59b4f1 
prod 반영(#351, #354)이 끝나 STG_BUSINESS_PREFIX 예외를 없앤다. stg 는 biz.members.* 도
CloudWatch 로 보내지 않는다. business_dashboard.py 는 stg 분기를 지우고 prod 만 그린다.
AWS 의 stg 대시보드 ruleup-stg-business 삭제는 따로 한다.

Refs #356

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
fix(notification): 약관 개정 고지는 대상 전체를 id 커서로 끝까지 돈다
21ba79b 
재동의하지 않은 사람은 매일 다시 대상이라, 앞 2,000명만 읽으면 뒤쪽은 고지를 영영 받지 못했다.
userId 커서로 쪽을 넘기며 한 실행에서 끝까지 돌고, 이미 받은 사람은 버전이 든 멱등 키가 걸러 준다.
실행 전체 트랜잭션은 걷어 내 쪽마다 적재가 커밋되게 했다. 쪽 크기는 설정값(기본 2,000)이다.

Fixes #377

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
fix(recommendation): 추천 이력 수집은 (확정 시각, id) 커서로 같은 시각을 넘어간다
8aa6e1a 
워터마크 시각 이상에서 앞 5,000건만 읽어, 같은 시각이 5,000건을 넘으면 매일 그 앞부분만 다시 읽고 멈췄다.
실행 안에서 (verifiedAt, id) 복합 커서로 쪽을 넘기며 끝까지 돌고, 쪽마다 자기 트랜잭션으로 커밋한다.
저장 워터마크는 기존 max(confirmedAt) 그대로 두고, 경계 시각 재수집·정정은 기존 upsert 가 흡수한다.

Fixes #383

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@selee1012 @claude
fix(verification): 원본 신호는 id 만 정렬하고 payload 는 PK 로 따로 읽는다
6e8b3ca 
prod 2026-10-30 sync 500 3건 — 「Out of sort memory」(1038). 한 날의 원본을 id 순으로 읽는 질의가
payload(JSON)까지 함께 정렬해, LIMIT 5,000 행분이 정렬 버퍼에 들어가야 했다.
정렬은 id 만으로 하고 payload·receivedAt 은 (observedDate, id) PK 로 읽어 id 순서대로 맞춘다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 30, 2026

Copy link
Copy Markdown

Test Coverage

There is no coverage information present for the Files changed

@github-actions

github-actions Bot commented Oct 30, 2026

Copy link
Copy Markdown

Test Coverage

Overall Project 83.97% -0.05% 🍏
Files changed 92.16% 🍏

File Coverage
TermsUpdateNoticeBatch.java 99.26% 🍏
SegmentScoreService.java 97.44% 🍏
ChallengeCleanupService.java 95.61% 🍏
CloudWatchMetricsConfig.java 95.48% 🍏
RoutineOutcomeCollector.java 87.81% -10.31% 🍏
SyncRateLimiter.java 87.62% -8.57% ❌
VerificationSyncService.java 87.07% 🍏
VerificationSignalReader.java 86.43% 🍏
ChallengeCreationService.java 84.05% 🍏
ImageStorageService.java 78.95% 🍏
AuthService.java 78.9% -0.31% 🍏
AuthController.java 71.43% 🍏
ChallengeSettingsService.java 70.04% -0.3% ❌
S3ImageObjectStore.java 63.87% -1.29% ❌
LocalImageObjectStore.java 62.07% -5.75% ❌

@selee1012
selee1012 merged commit d465bb9 into dev Oct 30, 2026
3 checks passed
@selee1012
selee1012 deleted the fix/quick-issues-2026-10-30 branch October 30, 2026 15:48
@selee1012 selee1012 mentioned this pull request Oct 30, 2026
Merged
selee1012 added a commit that referenced this pull request Oct 30, 2026
@selee1012
Merge pull request #387 from RuleUp-ASM/dev
21f044b 
release: JRE eMudhra 루트 제거 + 결정 불필요 이슈 일괄 수정 (#385 #386)
This was referenced Oct 31, 2026
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@selee1012