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

[ADD] 외부 API 예외 처리 #145

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

thguss
Copy link
Member

@thguss thguss commented Jan 17, 2024

✨ Related Issue

📝 기능 구현 명세

  • 외부 API에서 발생하는 예외를 처리합니다.

🐥 추가적인 언급 사항

none

@thguss thguss added add 작업 추가 sohyeon 소현 작업 labels Jan 17, 2024
@thguss thguss self-assigned this Jan 17, 2024
Copy link
Contributor

@csb9427 csb9427 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@@ -1,7 +1,10 @@
package com.soptie.server.auth.service;

import static com.soptie.server.auth.message.ErrorCode.*;
Copy link
Contributor

Choose a reason for hiding this comment

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

오 이거는 가독성을 위해 정적 메소드를 사용하신 건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 가독성과 통일성을 고려했습니다!

Copy link
Contributor

@Chan531 Chan531 left a comment

Choose a reason for hiding this comment

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

제 업무... 감사합니다...

@thguss thguss merged commit 760bb55 into develop Jan 17, 2024
1 check passed
@thguss thguss deleted the add/#126-auth-add-exception branch January 17, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add 작업 추가 sohyeon 소현 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ADD] auth 부분 예외처리
3 participants