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

에러 바운더리 적용 #40

Merged
merged 11 commits into from
Jul 29, 2023
Merged

에러 바운더리 적용 #40

merged 11 commits into from
Jul 29, 2023

Conversation

solar3070
Copy link
Owner

@solar3070 solar3070 commented Jul 29, 2023

🚩 관련 이슈

📋 작업 내용

  • 에러 바운더리 적용

📌 PR Point

[맞춤법 검사]

  • 맞춤법은 hanspell 라이브러리를 통해서 검사하고 있으며 부산대 맞춤법 검사기를 기본으로 사용합니다. 다만 종종 부산대 맞춤법 검사기 서버 접속 오류로 인해 에러가 발생하는 경우가 있는데 이럴 때는 다음 맞춤법 검사기를 통해 맞춤법을 검사합니다. 다음 맞춤법 검사기 또한 에러가 발생하는 경우 맞춤법 검사를 진행하지 않고 AI가 만든 문장을 그대로 사용자에게 보여줍니다.
Pasted Graphic

[AI 문장 생성]

  • Open AI API를 사용 중에 발생할 수 있는 오류는 다음과 같습니다. [링크]
  • 에러가 발생하면 에러의 발생 원인을 간단히 설명하고 잠시 후에 다시 시도하라는 안내 문구와 함께 Retry 버튼을 제공합니다.
  • 예외적으로 401에러나 429에러의 error code insufficient_quota 같은 경우는 유효하지 않은 인증 정보나 만료된 토큰으로 인해 발생하므로 사용자에게 다시하기 버튼을 제공하는 것이 의미가 없다고 판단하였습니다. 따라서 이 경우는 임시 문장을 제공하여 사용자가 아무것도 보지 못하는 상황이 발생하지 않도록 하였습니다.
Pasted Graphic 1
  • 추가적으로 Vercel의 hobby plan의 Serverless Function Execution Timeout10s이기 때문에 문장 생성 API를 serverless 함수에서 edge 함수로 변경하여 Timeout limit을 30s로 변경하였습니다. 다만, 에러가 발생할 수 있는 가능성은 여전히 있다고 생각하여 504에 대한 에러도 처리하였습니다. [관련 PR]

📸 스크린샷

다시하기 버튼 제공 임시 문장 제공
image image

@vercel
Copy link

vercel bot commented Jul 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
fixpace ✅ Ready (Inspect) Visit Preview Jul 29, 2023 7:53am

@solar3070 solar3070 self-assigned this Jul 29, 2023
@solar3070 solar3070 added the feat label Jul 29, 2023
@solar3070 solar3070 merged commit 030e937 into develop Jul 29, 2023
2 checks passed
@solar3070 solar3070 deleted the feat/#39 branch July 29, 2023 07:55
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.

에러 바운더리 적용
1 participant