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

좋아요 취소 기능 - JWT 인증 및 에러 처리 개선 #174

Merged
merged 3 commits into from
Mar 14, 2024

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Mar 14, 2024

작업 내용 (Content)

  • 좋아요 취소 기능을 사용자 인증과 연동했습니다.
    • cancelLike 서비스에서 사용자 ID 를 받는 방식을 엑세스 토큰으로 변경했습니다.
    • users/jwt/jwt.provider.ts 의 validateToken 함수를 사용하여 엑세스 토큰을 검증하고 사용자 ID 를 추출합니다.
    • cancel-like.controller.ts 에서 요청 헤더에서 엑세스 토큰을 가져오도록 수정했습니다.

링크 (Links)

기타 사항 (Etc)

Merge 전 필요 작업 (Checklist before merge)

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.
  • CI 파이프라인이 통과가 되었나요?
  • 리뷰 요청 전 Self-Review로 의문점을 해결 하셨나요?
  • PR 리뷰 가능한 크기를 유지하셨나요?

- 좋아요 취소 기능을 사용자 인증과 연동했습니다.
  - cancelLike 서비스에서 사용자 ID 를 받는 방식을 엑세스 토큰으로 변경했습니다.
  - users/jwt/jwt.provider.ts 의 validateToken 함수를 사용하여 엑세스 토큰을 검증하고 사용자 ID 를 추출합니다.
  - cancel-like.controller.ts 에서 요청 헤더에서 엑세스 토큰을 가져오도록 수정했습니다.
JWT 토큰 변경 및 장바구니 기능 JWT 인증 도입 및 테스트코드를 작성하라
- 좋아요 기능 호출 시 사용자 인증을 위해 accessToken 을 사용합니다.
  - users/jwt/jwt.provider.ts 의 validateToken 함수를 이용하여 토큰을 검증하고 사용자 ID 를 추출합니다.
좋아요 추가 로직을 간소화했습니다.

- 좋아요 도메인 객체 Like 를 직접 생성하지 않고 필요한 데이터 (userId, likedBookId)만 save 함수에 전달합니다.
@jihwooon jihwooon merged commit 4dca8be into main Mar 14, 2024
1 check passed
@jihwooon jihwooon deleted the issue-100 branch March 14, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant