Skip to content

fix: rt cookie 만료시간 테스트용 수정 - #219#220

Merged
sjk4618 merged 1 commit into
devfrom
fix/#219
Jan 18, 2026
Merged

fix: rt cookie 만료시간 테스트용 수정 - #219#220
sjk4618 merged 1 commit into
devfrom
fix/#219

Conversation

@sjk4618
Copy link
Copy Markdown
Member

@sjk4618 sjk4618 commented Jan 18, 2026

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

🚨 참고 사항

@sjk4618 sjk4618 self-assigned this Jan 18, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 18, 2026

📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

  • 개선 사항
    • 액세스 토큰 쿠키의 유효 기간을 2분에서 1주일로 연장했습니다. 사용자가 더 오래 로그인 상태를 유지할 수 있습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

쿠키 생성 유틸리티의 접근 토큰 최대 유효 시간 상수를 2분에서 7일로 변경했습니다. 쿠키 생성 메서드나 제어 흐름에는 변화가 없으며, 단순 상수값 업데이트만 수행되었습니다.

Changes

Cohort / File(s) 변경 요약
쿠키 유효 시간 상수 변경
src/main/java/com/permitseoul/permitserver/domain/auth/core/jwt/CookieCreatorUtil.java
COOKIE_MAX_AGE 상수값을 2L * 60 (2분)에서 7L * 24 * 60 * 60 (7일)로 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 이슈 #219는 테스트용 만료시간 설정 삭제를 요청하나, 실제 변경사항은 만료시간을 2분에서 7일로 변경한 것으로 요구사항과 불일치합니다. 이슈 #219의 요구사항을 확인하세요. 테스트용 시간 설정을 '삭제'해야 하는지 아니면 '변경'해야 하는지 명확히 하고, 필요시 코드 변경을 조정하세요.
Title check ❓ Inconclusive PR 제목이 변경사항과 관련이 있으나, 제목의 내용(테스트용 수정)과 실제 변경사항(쿠키 만료시간을 2분에서 1주일로 변경)이 완전히 일치하지 않습니다. 제목을 명확하게 수정하세요. 예: 'fix: access token 쿠키 만료시간을 1주일로 수정' 또는 테스트 관련이라면 실제 의도를 명시하세요.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed PR 설명이 링크된 이슈 #219와 관련이 있으며, 구체적인 작업 내용은 없지만 이슈 해결을 명시하고 있습니다.
Out of Scope Changes check ✅ Passed 변경사항이 단일 상수 값 수정만 포함하고 있으며, 링크된 이슈 범위 내 변경입니다.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

@sjk4618 sjk4618 merged commit 01d2ee3 into dev Jan 18, 2026
2 checks passed
@sjk4618 sjk4618 deleted the fix/#219 branch January 18, 2026 07:28
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.

test: at, rt 테스트용 시간 삭제

1 participant