Skip to content

patch: added exception handling for OpenAI requests#51

Merged
dongwang218 merged 1 commit intofacebookresearch:mainfrom
neuralsorcerer:patch-req
Aug 1, 2025
Merged

patch: added exception handling for OpenAI requests#51
dongwang218 merged 1 commit intofacebookresearch:mainfrom
neuralsorcerer:patch-req

Conversation

@neuralsorcerer
Copy link
Contributor

Changes:

  • Added exception handling for OpenAI requests so that errors from the OpenAI API are returned to clients with the correct status and detail.

Why?

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 1, 2025
Copy link
Contributor

@dongwang218 dongwang218 left a comment

Choose a reason for hiding this comment

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

Tested the pr and it work, thanks for the fix!

@dongwang218 dongwang218 merged commit 2b35950 into facebookresearch:main Aug 1, 2025
4 checks passed
@neuralsorcerer neuralsorcerer deleted the patch-req branch August 2, 2025 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

non descriptive Internal server error for openai badrequest exception

2 participants