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

♻️ [Refactor] 비정상 코드로 접근시 예외처리 / 인증없이 접근시 예외처리 #88

Merged
merged 3 commits into from
May 15, 2024

Conversation

88dldl
Copy link
Member

@88dldl 88dldl commented May 15, 2024

📍 PR 타입

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #87

🔁 변경 및 추가사항

dcdf985

  • 개발자 페이지 생성도 인증이 필요해서 허용된 url에서 제외했습니다

  • 예외처리
    인증없이 접근할 경우 예외처리 33b05c2
    비정상 코드로 인증 요청시 예외처리 95d4065

모두 BadRequest 로 처리해뒀습니다
image

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@88dldl 88dldl added the ♻️ Refactor 코드 리팩토링 label May 15, 2024
@88dldl 88dldl self-assigned this May 15, 2024
@88dldl 88dldl linked an issue May 15, 2024 that may be closed by this pull request
1 task
@88dldl 88dldl requested review from chanmin-00 and Ares-10 May 15, 2024 11:59
Copy link
Member

@chanmin-00 chanmin-00 left a comment

Choose a reason for hiding this comment

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

수고했어요!! 머지하면 테스트 해볼게요!!

@88dldl 88dldl merged commit 46aee04 into develop May 15, 2024
@88dldl 88dldl deleted the issue/87-refactor-error branch May 15, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

♻️ [Refactor] 인증 url 수정 및 서버 예외 처리 추가
2 participants