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: apple token 파싱 로직 base64, kid, alg 불일치 로그 메세지 구체화 #196

Merged
merged 1 commit into from
Sep 15, 2024

Conversation

Jaewon-pro
Copy link
Member

🔗 이슈 연결

🚀 구현한 API

  • X

💡 반영할 내용 및 변경 사항 요약

  • oidc jwt token parser에서 FAILED_PARSING 대신 INVALID_BASE64 에러 타입을 사용합니다.
  • kid, alg이 일치하는 경우가 없는 경우, 에러 메세지에 이 내용을 포함하도록 수정합니다.

🔍 리뷰 요청/참고 사항

Copy link
Collaborator

@hee9841 hee9841 left a comment

Choose a reason for hiding this comment

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

지금 db쪽 쿼리 에러등이 나도 클라이언트한테 e.getMessage()로 전체 메세지가 전달 되다보니, 쿼리문이 그대로 클라이언트한테 전달 되는 것 같아요!
에러 응답 메세지는 ErrorType의 message만 보내거나, 추가로 저희가 에러 메세지를 입력하면 ( e.getMessage() 말고 클라이언트에 전달되는 메시지) 해당 메시지 포함해서 응답으로 보내는 게 좋을 것 같아요!

상세 에러는 로그로 남기고요!

@Jaewon-pro
Copy link
Member Author

지금 db쪽 쿼리 에러등이 나도 클라이언트한테 e.getMessage()로 전체 메세지가 전달 되다보니, 쿼리문이 그대로 클라이언트한테 전달 되는 것 같아요!
에러 응답 메세지는 ErrorType의 message만 보내거나, 추가로 저희가 에러 메세지를 입력하면 ( e.getMessage() 말고 클라이언트에 전달되는 메시지) 해당 메시지 포함해서 응답으로 보내는 게 좋을 것 같아요!

상세 에러는 로그로 남기고요!

이건 이 PR과는 살짝 다른 내용인 것 같은데, 새로운 이슈에서 진행하면 좋을 것 같아요!

hee9841
hee9841 previously approved these changes Sep 15, 2024
Copy link
Collaborator

@hee9841 hee9841 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍

@Jaewon-pro Jaewon-pro merged commit 14e264e into main Sep 15, 2024
1 check passed
@Jaewon-pro Jaewon-pro deleted the feature/#195/apple-sign-id-token branch September 15, 2024 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sign in, sign up 요청 시 무조건 400 응답 반환되는 버그
2 participants