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

Api: ✏️ 총 지출 총합 값의 정수 범위 초과 케이스를 고려하여 타입 수정 #133

Merged
merged 15 commits into from
Jul 16, 2024

Conversation

psychology50
Copy link
Member

작업 이유


작업 사항

long 타입 수정하면서, 기존에 무분별하게 원시 타입과 래퍼 타입을 혼용하던 것을 다소 보완했습니다.

long 타입을 벗어나는 경우는 고려하지 않았습니다.
현실적으로 long 타입을 벗어날 정도가 되려면

2,147,483,647 x χ = 9,223,372,036,854,775,807
χ > 4,294,967,296

χ개 만큼의 소비 내역이 존재해야 하는데, 이 정도면 악의적인 공격이라고 밖에 볼 수 없으며, 시스템에 큰 지장을 준다고 생각도 안 되기에 별도의 예외 처리를 하지 않았습니다.

리뷰어가 중점적으로 확인해야 하는 부분

  • Wrapper Type과 Primitive Type을 혼용해서 사용하고 있는데, 타입 캐스팅으로 인한 오버헤드가 심각하게 발생할 만한 부분이 존재하는지?

발견한 이슈

당월 목표 금액 데이터가 없으면, 총 소비내역도 확인이 불가능한 이슈가 존재함.
하지만, 해당 API는 어디까지나 "당월" 목표 금액과 총 지출 금액을 조회하려는 용도로 개발되었으며, 정상적인 플로우에서 당월 목표 금액 데이터가 존재하지 않는 경우는 없음.
따라서 해당 이슈가 ciritcal하지 않다고 판단하여, 별도로 예외처리 하지 않음.

@psychology50 psychology50 added the fix 기능 수정 label Jul 16, 2024
@psychology50 psychology50 self-assigned this Jul 16, 2024
@psychology50
Copy link
Member Author

GPT 리뷰...똑똑한데 멍청해...돈 없으니까 적당히 리뷰 해줘...

@psychology50 psychology50 merged commit 127cf5c into dev Jul 16, 2024
1 check passed
@psychology50 psychology50 deleted the fix/PW-415-total-spending-type branch July 16, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 기능 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant