[4주차 과제] 이수정 #51
Open
[4주차 과제] 이수정 #51
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- SignUpRequest를 @RequestBody로 입력받아 validation을 검사 하고, 이상이 없으면 UserService의 signUp()으로 입력값을 전달해 이메일이 이미 가입되어 있는지 확인하고 가입되어 있지 않다면 비밀번호를 암호화하고 DB에 저장함 - DB에 입력받은 이메일이 존재하는 확인하기 위해 JPA 에 쿼리 메서드 existsByEmail(String email)을 정의함 - 휴대폰 번호 형식에 대한 validation 추가 - token을 사용할 예정이므로 시큐리티 설정 파일에 세션을 사용 하지 않는 설정과 CSRF 비활성화 추가
- enum으로 에러의 상태코드와 메시지를 정의하고 에러 발생 시 @RestControllerAdvice와 @ExceptionHandler를 사용해 처리되도록 코드 작성
sujeong11
requested review from
SeungYongChoi,
YOONSEOUNGHWAN and
won983212
as code owners
Collaborator
FeedBackFile.getOriginalFileName() method 에서 확장자를 제외하고 싶으신거라면 lastIndexoOf 등의 함수를 써서 subString으로 가져오는것이 가장 간단해 보이긴 합니다 :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이번 주에 어떤 작업을 했는지 설명해주세요.
특히 어떤 부분을 리뷰받고 싶나요?
s3Service의deleteFile메서드에서 사진 삭제를 구현 중 파일 경로 안에서 파일 이름만 가져올 때 substring()을 사용하고 있습니다. 혹시 파일 이름만 가져올 수 있는 더 나은 방법이 있는지 궁금합니다.이번 주는 어떻게 학습했나요? 아래 질문에 짧게 답변주세요!
이번 주에 학습에 투자한 시간
학습 하면서 좋았던 점과 아쉬웠던 점
좋았던 점
아쉬웠던 점
어려움을 겪는 부분
@RequestPart부분이 스웨거에 잘 적용되지 않는 것을 해결하지 못하고 있습니다.스터디 개선되었으면 하는 점