Skip to content

chore: jwt 설정 변경 및 스웨거 설정 수정#50

Merged
kon28289 merged 1 commit intodevfrom
chore/security
Dec 3, 2025
Merged

chore: jwt 설정 변경 및 스웨거 설정 수정#50
kon28289 merged 1 commit intodevfrom
chore/security

Conversation

@kon28289
Copy link
Copy Markdown
Contributor

@kon28289 kon28289 commented Dec 3, 2025

🚀 1. 개요

  • jwt의 secret-key와 accessToken의 유효기간을 변경합니다.
  • 개발 서버의 스웨거에서 잘못된 url로 요청을 보내는 오류를 수정합니다.

📝 2. 주요 변경 사항

  • 개발 서버의 accessToken의 유효기간을 10분, 배포 서버의 유효기간을 30분으로 변경합니다.

📸 3. 스크린샷 (API 테스트 결과)

Summary by CodeRabbit

Chores

  • 보안 관련 의존성이 최신 버전으로 업데이트되었습니다.

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

@kon28289 kon28289 requested a review from Juhye0k December 3, 2025 02:56
@kon28289 kon28289 self-assigned this Dec 3, 2025
@kon28289 kon28289 added the enhancement New feature or request label Dec 3, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 3, 2025

전체 요약

src/main/resources/security 서브모듈 참조가 커밋 e49128e에서 6851e1e로 업데이트되었습니다. 코드 또는 구조적 변경은 감지되지 않았습니다.

변경 사항

코호트 / 파일 변경 요약
서브모듈 업데이트
src/main/resources/security
서브모듈 참조를 e49128ebea530ac569fd10c309dfa8c5a195c597에서 6851e1e864ad98940081d668ae89424f7bd4cc52로 업데이트

코드 리뷰 예상 소요 시간

🎯 1 (Trivial) | ⏱️ ~2분

추천 검수자

  • Juhye0k

시 🐰

조용한 밤, 보안 벽 너머
새 커밋이 살금살금 들어오네
버전 번호만 바뀌고, 논리는 그대로
안전한 업데이트, 모두를 위해! 🔐✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 JWT 설정 변경과 스웨거 설정 수정이라는 주요 변경 사항을 명확히 요약하고 있으며, raw_summary의 보안 모듈 업데이트 내용과 일치합니다.
Description check ✅ Passed PR 설명이 템플릿의 세 가지 필수 섹션(개요, 주요 변경 사항, 스크린샷)을 모두 포함하고 있으며, 처음 두 섹션은 충분히 작성되었습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/security

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 82c51a8 and 8dbdeb3.

📒 Files selected for processing (1)
  • src/main/resources/security (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-12-01T03:06:04.769Z
Learnt from: Juhye0k
Repo: Geumpumta/backend PR: 41
File: src/main/resources/security:1-1
Timestamp: 2025-12-01T03:06:04.769Z
Learning: The `backend-security` repository at https://github.com/Geumpumta/backend-security.git is a private repository used as a submodule at src/main/resources/security, so automated verification of submodule commits may fail due to access restrictions.

Applied to files:

  • src/main/resources/security
🔇 Additional comments (1)
src/main/resources/security (1)

1-1: 서브모듈 변경 사항 검증 필요 - 보안 민감도 높음

backend-security 서브모듈 참조가 업데이트되었습니다. PR 목표에서 언급한 JWT 설정 변경 및 스웨거 설정 수정이 이 커밋(6851e1e864ad98940081d668ae89424f7bd4cc52)에 포함되어 있는지 확인이 필요합니다.

서브모듈이 비공개 저장소이므로, 다음 사항을 직접 검증해주시기 바랍니다:

  1. JWT 시크릿 키 변경 사항 확인
  2. AccessToken 유효성 기간 조정 확인 (개발: 10분, 운영: 30분)
  3. 스웨거 URL 설정 수정 사항 확인
  4. 의도하지 않은 변경 사항 포함 여부 확인

또한 서브모듈 커밋의 커밋 메시지와 커밋 서명도 함께 검토해주시기 바랍니다.


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.

Copy link
Copy Markdown
Contributor

@Juhye0k Juhye0k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kon28289 kon28289 merged commit bca3567 into dev Dec 3, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants