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.

Sign up for GitHub

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

Jump to bottom

Raise invalid request exceptions #581

Merged
merged 5 commits into from
Apr 1, 2024
Merged

Raise invalid request exceptions #581

merged 5 commits into from
Apr 1, 2024

Conversation

eyurtsev
Copy link
Collaborator

@eyurtsev eyurtsev commented Apr 1, 2024 •
edited
Loading

  • This PR removes dependency on httpx sse
  • After this PR invalid requests will be returned as having a status code of 422 using a non streaming request

@cla-bot cla-bot bot added the cla-signed label Apr 1, 2024
@eyurtsev eyurtsev changed the title Raise invalid request exceptions LangServe: Raise invalid request exceptions Apr 1, 2024
@eyurtsev eyurtsev requested a review from nfcampos April 1, 2024 16:46
@eyurtsev eyurtsev merged commit 014acfc into main Apr 1, 2024
11 checks passed
@eyurtsev eyurtsev deleted the eugene/api_handler_2 branch April 1, 2024 16:52
@eyurtsev eyurtsev changed the title LangServe: Raise invalid request exceptions Raise invalid request exceptions Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nfcampos nfcampos Awaiting requested review from nfcampos

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@eyurtsev