fix: 활성 공유 목록에서 만료된 공유 제외 - #163
Merged
Merged
Conversation
만료는 상태가 아니라 expires_at 비교로 판정하는 것이 이 도메인의 설계다. 목록 조회가 status 컬럼만 보고 있어 이미 열리지 않는 링크가 활성으로 잡혔다.
This was referenced Sep 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
관련 이슈
변경 내용
ShareRepositoryPort.findPage에 만료 판정 기준 시각(now) 파라미터 추가status=ACTIVE필터에expires_at > now조건 추가작업 목적
만료는 상태가 아니라
expires_at비교로 판정하는 것이 이 도메인의 설계인데(V19 주석),목록 조회만
status컬럼을 보고 있어 이미 열리지 않는 링크가 활성으로 잡혔다.공개 조회는 만료를 검사하므로 목록에는 살아있는데 실제로 열면 실패하는 링크가 생긴다.
테스트 방법
SharingPersistenceAdapterIntegrationTest에 네 경우를 고정했다 — 만료 제외 / 만료시각이 현재와 같을 때 / 전체 조회는 유지 / REVOKED 필터는 유지.
체크리스트
기타 사항
GET /shares?status=ACTIVE의 결과 집합이 좁아진다. 응답의status필드는 저장된값을 그대로 내려주므로 스키마 변경은 없다.
아닌 설계의 귀결이라 그대로 뒀다.