Skip to content

fix: 인증 사용자 정보 사용 수정 (#64) - #65

Merged
young0206 merged 1 commit into
developfrom
refactor/64-video
Jul 24, 2026
Merged

fix: 인증 사용자 정보 사용 수정 (#64)#65
young0206 merged 1 commit into
developfrom
refactor/64-video

Conversation

@young0206

@young0206 young0206 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

🔗 관련 이슈 (Related Issue)

Closes #64

📝 작업 내용

  • 영상 API에서 임시로 사용하던 memberId = 1L을 제거했습니다.
  • 인증된 사용자 정보를 기반으로 memberId를 조회하도록 변경했습니다.
  • 관련 영상 API가 인증된 사용자 기준으로 정상 동작하도록 수정했습니다.

✅ PR 체크리스트

  • PR 제목은 커밋 컨벤션을 따랐습니다.
  • 관련 이슈를 연결했습니다.
  • 변경 사항에 대한 테스트를 진행했습니다.

Summary by CodeRabbit

  • 새 기능
    • 영상 생성·조회·수정·삭제 및 YouTube URL 검증 시 로그인한 사용자의 인증 정보를 적용합니다.
  • 버그 수정
    • 영상 관련 API에서 특정 사용자로 고정되던 문제를 개선했습니다.
  • 문서
    • Swagger API 설명, 요청·응답 필드 설명 및 예시를 보강했습니다.
    • 파라미터 입력 검증과 API 문서 표시를 정리했습니다.

@young0206 young0206 self-assigned this Jul 23, 2026
@young0206 young0206 added the refactor 기능 변경 없는 코드 개선 label Jul 23, 2026
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ef200bd5-8c99-4a28-aaaa-9bb39331d708

📥 Commits

Reviewing files that changed from the base of the PR and between 2bdee86 and aebefe8.

📒 Files selected for processing (4)
  • src/main/java/com/slatto/domain/video/controller/VideoController.java
  • src/main/java/com/slatto/domain/video/controller/YoutubeController.java
  • src/main/java/com/slatto/domain/video/dto/request/VideoRequest.java
  • src/main/java/com/slatto/domain/video/dto/response/VideoResponse.java

📝 Walkthrough

Walkthrough

영상 및 YouTube API가 인증된 사용자의 memberId를 사용하도록 변경되었으며, 컨트롤러와 DTO의 OpenAPI 문서 메타데이터가 갱신되었습니다.

Changes

영상 API 인증 및 문서화

Layer / File(s) Summary
인증 사용자 식별자 주입
src/main/java/com/slatto/domain/video/controller/VideoController.java, src/main/java/com/slatto/domain/video/controller/YoutubeController.java
영상 생성·조회·수정·삭제 및 YouTube URL 검증 엔드포인트가 @AuthenticationPrincipal Long memberId를 사용하도록 변경되었습니다.
OpenAPI 문서와 DTO 메타데이터 갱신
src/main/java/com/slatto/domain/video/controller/*Controller.java, src/main/java/com/slatto/domain/video/dto/request/VideoRequest.java, src/main/java/com/slatto/domain/video/dto/response/VideoResponse.java
컨트롤러 태그·파라미터 설명과 요청·응답 필드의 Schema 설명 및 예시가 갱신되었습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • SLAT-TO/SLAT-TO-BE#40: 컨트롤러의 사용자 식별자를 하드코딩 또는 헤더에서 @AuthenticationPrincipal 주입 방식으로 변경한 PR입니다.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 인증 사용자 정보로 memberId를 사용하는 핵심 변경을 정확히 요약합니다.
Description check ✅ Passed 필수 섹션인 관련 이슈, 작업 내용, 체크리스트가 모두 포함되어 있습니다.
Linked Issues check ✅ Passed 하드코딩된 memberId 제거와 인증 사용자 기반 주입으로 #64의 요구사항을 충족합니다.
Out of Scope Changes check ✅ Passed Swagger/DTO 주석 정리도 영상 API 변경과 직접 연관되어 있어 별도 범위 이탈로 보이지 않습니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@young0206
young0206 merged commit 8e71df4 into develop Jul 24, 2026
2 checks passed
@young0206
young0206 deleted the refactor/64-video branch July 24, 2026 12:10
@coderabbitai coderabbitai Bot mentioned this pull request Jul 24, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 기능 변경 없는 코드 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

REFACTOR: 영상 API 인증 사용자 정보 사용으로 변경

2 participants