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.

Sign up for GitHub

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

Jump to bottom

Feature/soo step1 #2

Merged
merged 66 commits into from
Feb 26, 2024
Merged

Feature/soo step1 #2

merged 66 commits into from
Feb 26, 2024

Conversation

youngsoosoo
Copy link

@youngsoosoo youngsoosoo commented Jan 29, 2024 •
edited
Loading

ERD

image

회원 가입 및 인증

사용자 정보는 평범하게 구현했습니다.

  • 로그인: 로그인 시 회원의 기본키 정보가 담긴 JWT 토큰을 반환하도록 진행
  • 회원가입: 회원가입 시 회원의 계좌를 생성해주기 위해 이벤트 리스너 등록

개선점

  • 로그인 시 하나의 JWT 토큰이 아닌 RefreshToken, AccessToken으로 나누어 보안성을 높이는 것이 좋을 거 같습니다.

계좌

  • 계좌 Type: Enum을 통해 계좌의 형태를 나타내주었습니다.
  • 1일 충전 한도: 관계형 데이터베이스에 컬럼을 추가해 관리할 수 있도록 만들었습니다.
  • 계좌 간 거래: 특정 엔티티(행)에 대한 동시성 제어를 위해 비관적 락 사용, 메인 계좌에 잔액이 부족할 시 에러 처리
    • 트랜잭션 격리 수준 중 SERIALIZABLE 수준과 REPEATABLE_READ 수준을 적용하면 데드락이 발생합니다. 그래서 READ_COMMITED 수준과 락을 통해 동시성 제어를 해주었습니다.

개선점

  • 현재는 1일 충전 한도를 관계형 데이터 베이스에 저장하고 있습니다. 이후에 Redis의 TTL(Time To Live) 이라는 기능을 통해 관리해주면 좋을 거 같습니다.

@youngsoosoo
chore: DB, Swagger, Security 의존성 추가, application.yml 설정 추가
a1c3d30
@youngsoosoo
chore: JPA 의존성 추가
e84318d
@youngsoosoo
feat: BaseEntity 및 Status 작성, JPA Auditing 기능 설정
a70e910
@youngsoosoo
feat: Member Entity 작성
8c4f47b
@youngsoosoo
feat: Account Entity 작성, Type Enum 작성, Status 속성 추가
ac00749
@youngsoosoo
fix: Account Builder 잔액은 0으로 생성하고, type은 생성 시 설정할 수 있도록 변경
d220234
@youngsoosoo
feat: 예외 처리를 위한 ErrorCode 및 BaseException 작성, GlobalExceptionHandler 작성
2f2a347
@youngsoosoo
feat: MemberRepository 추가
21bafef
@youngsoosoo
feat: 회원가입과 로그인을 위한 메서드 선언
08f56a4
@youngsoosoo
test: 로그인, 회원가입 서비스 테스트 코드 작성
d850421
@youngsoosoo
refactor: 예외 처리 변경
20663c7
@youngsoosoo
chore: JWT 의존성 추가
ef834d2
@youngsoosoo
feat: JWT 토큰 관련 코드 작성, Security 코드 작성
7d0a204
@youngsoosoo
feat: 회원가입 코드 작성
bf81b7a
@youngsoosoo
feat: 로그인 코드 추가
5a7b7f9
@youngsoosoo
fix: 순환 종속성 문제 해결
7e52863
@youngsoosoo
test: MockMVC를 통해 Controller 테스트 코드 작성
a5dfdd6
@youngsoosoo
feat: 토큰을 검사하고 회원 정보를 출력하는 코드 추가
d5dd5fa
@youngsoosoo
feat: Account Entity에 일일 한도 컬럼 추가
9162f9f
@youngsoosoo
feat: Account Repository 추가
8c2acf3
@youngsoosoo
fix: 1일 한도 컬럼명 및 Builder 수정
e87a3a5
@youngsoosoo
feat: DailyLimit를 초기화 하기 위한 메서드 생성
c957f03
@youngsoosoo
test: 계좌 관련 테스트 코드 작성
b258086
@youngsoosoo
feat: 계좌 생성 메서드 추가
e8d1c78
@youngsoosoo
fix: JwtTokenUtil을 통해 회원 정보 가져오기
ec1790d
@youngsoosoo
fix: 헤더로 토큰 전송하는 방법으로 변경
0eb75e3
@youngsoosoo
feat: 계좌 생성 API 코드 추가
9f2ddc8
@youngsoosoo
feat: 회원의 계좌 정보 조회 코드 추가(동시성 제어를 위해 행 수준 잠금), 잔액 수정 코드 추가
db2ca30
@youngsoosoo
test: 사용자의 계좌 조회 테스트 코드 작성, 사용자의 메인 계좌로 이체되는 코드 작성
9a59ca5
@youngsoosoo
test: 회원 정보로 계좌를 조회하도록 수정
06dac6b
@youngsoosoo youngsoosoo self-assigned this Feb 17, 2024
@youngsoosoo youngsoosoo added the enhancement New feature or request label Feb 17, 2024
@youngsoosoo
refactor: 코드 리뷰 반영
512f777 
- 시큐리티 CORS 설정 변경, swagger 사용을 위해 permitAll url 추가

- @Email로 변경

- ErrorCode 수정(HTTP STATUS CODE를 올바르게 수정)
@youngsoosoo
test: HTTP STATUS 수정
b9c9d22
@youngsoosoo
feat: Account Entity에 인덱스 설정
cd09f11
@youngsoosoo
refactor: 토큰 저장 값 변경
e5f46fb 
- 토큰 생성 시 memberId를 저장하고 이를 통해 member 데이터에 접근 단축
@youngsoosoo
test: 토큰 저장 값 변경
775c9fa 
- 토큰 생성 시 memberId를 저장하고 이를 통해 member 데이터에 접근을 단축
@youngsoosoo
refactor: 계좌 생성 시 메인 계좌가 유무 확인, 계좌 이체 로직 순서 변경
88a3619
@youngsoosoo
test: 계좌 생성 시 메인 계좌 유무 확인 테스트 작성
da6bbe4
@youngsoosoo
fix: 트랜잭션 격리 수준 변경(REPEATABLE_READ -> READ_COMMITED))
a6fd8ee
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
11 New issues

Measures
0 Security Hotspots
82.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@youngsoosoo youngsoosoo requested a review from VSFe February 22, 2024 18:44
@youngsoosoo youngsoosoo merged commit a771d96 into base/soo Feb 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yunsik0115 yunsik0115 Awaiting requested review from yunsik0115

@kariskan kariskan Awaiting requested review from kariskan

@seungh1024 seungh1024 Awaiting requested review from seungh1024

@VSFe VSFe Awaiting requested review from VSFe

Assignees

@youngsoosoo youngsoosoo

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@youngsoosoo @VSFe @seungh1024