Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] #193 비디오 도메인의 오류를 수정했어요 #194

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

BaeJunH0
Copy link
Contributor

✨ 작업 내용

  • influencer 정보를 이용하여 조회하는 부분의 논리적인 오류 ( id 오름차순 정렬이라서 계속 같은 값만 나올 것 ) 해결
  • videos/my에 토큰 검증 추가

✨ 참고 사항


⏰ 현재 버그


✏ Git Close

BaeJunho added 2 commits November 15, 2024 02:00
토큰 검증이 필요한 부분에 토큰 검증 추가
@BaeJunH0 BaeJunH0 added the 🐞 BugFix Something isn't working label Nov 14, 2024
@BaeJunH0 BaeJunH0 self-assigned this Nov 14, 2024
// 토큰 정보에 대한 검증
if (AuthorizationUtil.isNotLoginUser()) {
throw InplaceException.of(AuthorizationErrorCode.TOKEN_IS_EMPTY);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 안해도 자동으로 로그인 유저 아니면 막아주긴 합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 그래요?

@BaeJunH0 BaeJunH0 merged commit ecbe522 into weekly/11 Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants