Skip to content

fix: 활성 공유 목록에서 만료된 공유 제외 - #163

Merged
redjungi09 merged 2 commits into
developfrom
fix/active-share-expiry/#162
Sep 3, 2026
Merged

fix: 활성 공유 목록에서 만료된 공유 제외#163
redjungi09 merged 2 commits into
developfrom
fix/active-share-expiry/#162

Conversation

@redjungi09

Copy link
Copy Markdown
Collaborator

관련 이슈

변경 내용

  • ShareRepositoryPort.findPage 에 만료 판정 기준 시각(now) 파라미터 추가
  • status=ACTIVE 필터에 expires_at > now 조건 추가
  • REVOKED 필터·전체 조회는 만료로 거르지 않음
  • Swagger 설명에 ACTIVE 필터의 만료 기준 명시

작업 목적

만료는 상태가 아니라 expires_at 비교로 판정하는 것이 이 도메인의 설계인데(V19 주석),
목록 조회만 status 컬럼을 보고 있어 이미 열리지 않는 링크가 활성으로 잡혔다.
공개 조회는 만료를 검사하므로 목록에는 살아있는데 실제로 열면 실패하는 링크가 생긴다.

테스트 방법

  • 단위 테스트 추가/수정
    • 통합 테스트 완료
    • 수동 테스트 완료

SharingPersistenceAdapterIntegrationTest 에 네 경우를 고정했다 — 만료 제외 / 만료
시각이 현재와 같을 때 / 전체 조회는 유지 / REVOKED 필터는 유지.

체크리스트

  • 코드가 프로젝트의 코딩 스타일을 따릅니다
    • 자체 코드 리뷰를 완료했습니다
    • 변경 사항에 대한 테스트를 추가했습니다
    • Breaking Changes가 없습니다

기타 사항

  • GET /shares?status=ACTIVE 의 결과 집합이 좁아진다. 응답의 status 필드는 저장된
    값을 그대로 내려주므로 스키마 변경은 없다.
  • 만료된 링크는 어느 상태 필터에도 잡히지 않고 전체 조회에서만 보인다. 만료가 상태가
    아닌 설계의 귀결이라 그대로 뒀다.

만료는 상태가 아니라 expires_at 비교로 판정하는 것이 이 도메인의 설계다.
목록 조회가 status 컬럼만 보고 있어 이미 열리지 않는 링크가 활성으로 잡혔다.
@redjungi09
redjungi09 merged commit 737202b into develop Sep 3, 2026
1 check passed
@redjungi09
redjungi09 deleted the fix/active-share-expiry/#162 branch September 3, 2026 11:13
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