Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

회원가입 기능 #3

Closed
choitaehoon opened this issue Sep 25, 2020 · 0 comments
Closed

회원가입 기능 #3

choitaehoon opened this issue Sep 25, 2020 · 0 comments
Assignees

Comments

@choitaehoon
Copy link
Collaborator

choitaehoon commented Sep 25, 2020

  • 예외 반환 설계 논의 후 프로덕션 코드 및 테스트 코드 구현
@choitaehoon choitaehoon self-assigned this Sep 25, 2020
choitaehoon added a commit that referenced this issue Sep 25, 2020
- 회원가입 구현, 테스트 코드 작성
- 추후에 Global Exception Handling 검토
choitaehoon added a commit that referenced this issue Oct 8, 2020
* [#3] 회원가입 기능

- 회원가입 구현, 테스트 코드 작성
- 추후에 Global Exception Handling 검토

* 회원가입 리뷰 반영

- AccountController의 accountRegister api 이름 변경
- ResponseEntity 리턴 값 상수 적용
- NoArgsConstructor(AccessLevel.PROTECTED) 적용
- application.yml 로컬/운영 환경설정 분리
- 암호화 확장

* 회원가입 리뷰 반영

- ResponseEntity 리턴 값 명확한 이름으로 변경
- ResponseEntity error, success 분리(유지보수성)
- Exception Handling 적용
- EncryptionFactory bean 등록
- 추후에 sha256Util, sha256Service naming 결정

* 회원가입 리뷰 반영

- CheckStyle 적용
- ResponseConstants naming 변경

* 회원가입 리뷰 반영

- ResponseConstants로 naming 변경

* 회원가입 기능

- AccountDto message 수정
- MethodArgumentNotValidException handler 추가
- ErrorMessage 생성

* 회원가입 기능

- ErrorMessage 가독성 증가 및 부연 설명

* 회원가입 기능

- ErrorMessage 줄바꿈 문자 상수 적용

* 회원가입 기능

- EncryptionEncoding, Sha256Encoding으로 naming 변경
- Encoding에 관한 상수 분리
- ErrorMessage 주석 수정

* 회원가입 리뷰 반영

- 암호화 naming Encryption으로 변경
choitaehoon added a commit that referenced this issue Oct 8, 2020
* [#3] 회원가입 기능

- 회원가입 구현, 테스트 코드 작성
- 추후에 Global Exception Handling 검토

* 회원가입 리뷰 반영

- AccountController의 accountRegister api 이름 변경
- ResponseEntity 리턴 값 상수 적용
- NoArgsConstructor(AccessLevel.PROTECTED) 적용
- application.yml 로컬/운영 환경설정 분리
- 암호화 확장

* 회원가입 리뷰 반영

- ResponseEntity 리턴 값 명확한 이름으로 변경
- ResponseEntity error, success 분리(유지보수성)
- Exception Handling 적용
- EncryptionFactory bean 등록
- 추후에 sha256Util, sha256Service naming 결정

* 회원가입 리뷰 반영

- CheckStyle 적용
- ResponseConstants naming 변경

* 회원가입 리뷰 반영

- ResponseConstants로 naming 변경

* 회원가입 기능

- AccountDto message 수정
- MethodArgumentNotValidException handler 추가
- ErrorMessage 생성

* 회원가입 기능

- ErrorMessage 가독성 증가 및 부연 설명

* 회원가입 기능

- ErrorMessage 줄바꿈 문자 상수 적용

* 회원가입 기능

- EncryptionEncoding, Sha256Encoding으로 naming 변경
- Encoding에 관한 상수 분리
- ErrorMessage 주석 수정

* 회원가입 리뷰 반영

- 암호화 naming Encryption으로 변경
choitaehoon added a commit that referenced this issue Oct 14, 2020
* [#3] 회원가입 기능

- 회원가입 구현, 테스트 코드 작성
- 추후에 Global Exception Handling 검토

* 회원가입 리뷰 반영

- AccountController의 accountRegister api 이름 변경
- ResponseEntity 리턴 값 상수 적용
- NoArgsConstructor(AccessLevel.PROTECTED) 적용
- application.yml 로컬/운영 환경설정 분리
- 암호화 확장

* 회원가입 리뷰 반영

- ResponseEntity 리턴 값 명확한 이름으로 변경
- ResponseEntity error, success 분리(유지보수성)
- Exception Handling 적용
- EncryptionFactory bean 등록

* [#5] 로그인/로그아웃 구현

- 로그인시 세션 생성, 로그아웃시 세션 소멸
- 로그인시 유저에 대한 정보가 없으면 exception handling

* 로그인/로그아웃 리뷰 반영

- session key 상수화
- 로그인 관련 세션 데이터 입출력 책임 분리

* 로그인/로그아웃 리뷰 반영

- session 값 입출력 책임 분리
- session 상수 naming 변경
- hasAccount -> checkHasAccount 메소드명 변경
- 추후에 session 리팩토링(session이 늘어남에 따라 각자에 맞는 세션 생성되므로 메소드 증가, OCP 적용 검토)

* 로그인/로그아웃 리뷰 반영

- 유효성, session method naming 변경
- SessionService 사용하는 이유 주석 작성
- SessionKey Account -> USER_ID naming으로 변경

* 로그인/로그아웃 리뷰 반영

- SessionService를 분리 한 이유 주석 수정

* 로그인/로그아웃 리뷰 반영

- SessionService 캡슐화 부연설명 추가
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant