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

fix: 헤더가 빈 값이면 NPE 발생 #932

Merged
merged 1 commit into from
Dec 28, 2024
Merged

fix: 헤더가 빈 값이면 NPE 발생 #932

merged 1 commit into from
Dec 28, 2024

Conversation

mzeong
Copy link
Member

@mzeong mzeong commented Dec 23, 2024

🚩 연관 이슈

close #622


📝 작업 내용

헤더가 널일 때 NPE가 발생해 500이 반환되어 안드 측에서 에러 확인 문의가 반복적으로 일어나 해결합니다


🏞️ 스크린샷 (선택)


🗣️ 리뷰 요구사항 (선택)

@mzeong mzeong self-assigned this Dec 23, 2024
Copy link

Test Results

 63 files   63 suites   13s ⏱️
209 tests 209 ✅ 0 💤 0 ❌
210 runs  210 ✅ 0 💤 0 ❌

Results for commit eb38cd1.

Copy link

📝 Test Coverage Report

Overall Project 82.79% 🍏
Files changed 100% 🍏

File Coverage
AccessToken.java 100% 🍏
RefreshToken.java 64.84% 🍏

Copy link
Contributor

@coli-geonwoo coli-geonwoo left a comment

Choose a reason for hiding this comment

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

크게 리뷰할 부분이 많지 않아 approve 합니다! 고생많았어요 제리~

Copy link
Contributor

@eun-byeol eun-byeol left a comment

Choose a reason for hiding this comment

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

리뷰가 늦었네요! 고생했어요 제리~🐹

@eun-byeol eun-byeol merged commit 6dbedfc into develop Dec 28, 2024
3 checks passed
@eun-byeol eun-byeol deleted the feature/622 branch December 28, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 헤더가 빈 값이면 NPE 발생
3 participants