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

feat: 문자 인증 기능 추가 #45

Merged
merged 17 commits into from
Jan 19, 2024

Conversation

seokho-1116
Copy link
Collaborator

작업 내용 (Content)

  • 문자 인증 엔드포인트 추가
  • Api 요청 제한 추가

링크 (Links)

기타 사항 (Etc)

  • RestTemplate에 대한 설정이 추가적으로 필요하다. 링크 참조
  • 소켓을 SSL에서 가져오는지 확인 했음Screenshot from 2024-01-19 12-50-15
  • keep-alive 타임도 확인 후 config에 설정
    Screenshot from 2024-01-19 12-50-38
  • RestTemplate에선 내가 쓴 요청이나 반환된 응답을 Http Converter가 동작해서 변환함. 해당 변환기에 대한 설정도 필요.

Merge 전 필요 작업 (Checklist before merge)

희망 리뷰 완료 일 (Expected due date)

- 문자 발송 엔드포인트에 대한 컨트롤러 추가
- Socket 커넥션 풀을 이용하기 위해 Apache 의존성 추가
- 외부 API 예외 추가
- API 요청 제한 예외 추가
- 문자 인증 request 추가
- 문자 인증 response 추가
Open #29
- 4자리 인증 번호 전송 서비스 추가
Open #29
- 인증 문자 전송 내역 캐시 레포지토리 추가
Open #29
- 알리고 문자 인증 전송 서비스 추가
- 관련 프로퍼티 추가
- 응답 추가
Open #29
- 문자 인증 횟수 제한 인터셉터 추가
- Api 사용 횟수 캐시 레포지토리 추가
- 인터셉터 path config 추가
- auth/dto 아래로 변경
- rest template 관련 설정 추가
Open #29
- core 하위에 필요 없는 .github 삭제
@seokho-1116 seokho-1116 added the feat 새로운 기능 label Jan 19, 2024
@seokho-1116 seokho-1116 self-assigned this Jan 19, 2024
- 인증 메시지 전송 후 인증 객체 저장하도록 수정
- Custom Validator & Validation 추가
- RestTemplate 설정 변경
- 변수 -> 상수화
Copy link
Collaborator

@GaBaljaintheroom GaBaljaintheroom left a comment

Choose a reason for hiding this comment

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

@GaBaljaintheroom GaBaljaintheroom merged commit 88d77c4 into develop_back Jan 19, 2024
2 checks passed
@seokho-1116 seokho-1116 deleted the ARCH-114-feat/message-verification branch January 25, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants