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

[3주차 과제] 이수정 #40

Open
wants to merge 29 commits into
base: main
Choose a base branch
from

Conversation

sujeong11
Copy link

이번 주에 어떤 작업을 했는지 설명해주세요.

  • swagger 설정 후 문서화 코드 추가
  • S3를 이용한 다중 이미지 업로드 구현
  • 상품 등록 API
  • 좋아요 추가 / 삭제 API
  • 인터셉터를 이용해 응답 값 통일

특히 어떤 부분을 리뷰받고 싶나요?

  1. S3 다중 이미지 업로드 코드 부분입니다. 이미지들을 S3 버킷에 올리면서 File 엔티티에도 파일 이름이랑 url을 저장하고 있는데 이 부분에서 제가 작성한 코드가 잘 작성된건지 궁금합니다.
  2. Dto 안에 있는 필드의 타입이 엔티티일 때 어떤 식으로 처리하는 것이 좋을지 궁금합니다.
public class ProductInfoResponseDto {

   private User user;
   ...
}

이번 주는 어떻게 학습했나요? 아래 질문에 짧게 답변주세요!

이번 주에 학습에 투자한 시간

  • 25시간

학습 하면서 좋았던 점과 아쉬웠던 점

  • 좋았던 점: S3에 이미지 업로드를 하는 방법을 알게된 것
  • 아쉬웠던 점: 아직 스프링부트를 잘 다루지 못해 더 많은 API를 구현하지 못한 것

어려움을 겪는 부분

  • 제공해주신 자료 참고해서 인터셉터를 구현하고 실행해보니 JwtAuthenticationFilter.java의 doFilter에서 토큰 검증할 때 SecurityException와 같은 에러 발생 시 catch문에 걸리지 않는데 어디가 문제인지 잘 모르겠습니다..

스터디 개선되었으면 하는 점

  • 없습니다.

sujeong11 and others added 29 commits January 18, 2023 18:36
- SignUpRequest를 @RequestBody로 입력받아 validation을 검사 하고, 이상이 없으면 UserService의 signUp()으로 입력값을 전달해 이메일이 이미 가입되어 있는지 확인하고 가입되어 있지 않다면  비밀번호를 암호화하고 DB에 저장함
- DB에 입력받은 이메일이 존재하는 확인하기 위해 JPA 에 쿼리 메서드 existsByEmail(String email)을 정의함
- 휴대폰 번호 형식에 대한 validation 추가
- token을 사용할 예정이므로 시큐리티 설정 파일에 세션을 사용 하지 않는 설정과 CSRF 비활성화 추가
- enum으로 에러의 상태코드와 메시지를 정의하고 에러 발생 시 @RestControllerAdvice와 @ExceptionHandler를 사용해 처리되도록 코드 작성
@sujeong11 sujeong11 requested a review from won983212 as a code owner February 5, 2023 09:30
@won983212
Copy link
Collaborator

  1. 네 잘 작성해주신 것 같습니다.
  2. Dto로 한번 감쌌다고 하더라도 Entity를 바로 반환하는 건 여러 위험요소들이 있습니다. 그래서 저는 가능한 모두 Dto로 만들어서 사용합니다. 그래서 User타입을 Dto를 따로 만들어서 사용할 것 같습니다.

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.

2 participants