-
Notifications
You must be signed in to change notification settings - Fork 2
fix: 탈퇴한 사용자와 채팅 시도 시 발생하는 NPE 수정 #2132
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
Conversation
dh2906
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니당
| return Optional.ofNullable(lostItemArticle.getAuthor()) | ||
| .map(User::getProfileImageUrl) | ||
| .orElseThrow(() -> CustomException.of(ApiResponseCode.NOT_FOUND_CHAT_PARTNER)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
유저의 profileImageUrl이 null인 경우는 404가 터지는 로직 같은데, 현재 거의 모든 유저는 이미지가 없어서 거의 무조건 터지지 않나요?? 👀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
헐 다 null로 들어가있네요
kih1015
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다~
🔍 개요
🚀 주요 변경 내용
{ "message": "분실물 게시글 채팅방이 존재하지 않습니다.", "errorTraceId": "123e4567-e89b-12d3-a456-426614174000", "code": "NOT_FOUND_CHAT_PARTNER" }💬 참고 사항
✅ Checklist (완료 조건)