[PR #133] [MERGED] Dynamic validation for streaming endpoints #369

Closed
opened 2026-02-16 00:20:19 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langserve/pull/133
Author: @eyurtsev
Created: 10/30/2023
Status: Merged
Merged: 10/31/2023
Merged by: @eyurtsev

Base: mainHead: eugene/add_input_validation_streaming


📝 Commits (6)

📊 Changes

4 files changed (+663 additions, -314 deletions)

View changed files

📝 langserve/server.py (+329 -194)
📝 langserve/validation.py (+31 -0)
📝 tests/unit_tests/test_server_client.py (+298 -117)
📝 tests/unit_tests/test_validation.py (+5 -3)

📄 Description

  • Add test to verify that client side errors are now raised for streaming endpoints
  • Add more tests to verify that config and inputs are being validated properly -- or introduce more pydantic models

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langserve/pull/133 **Author:** [@eyurtsev](https://github.com/eyurtsev) **Created:** 10/30/2023 **Status:** ✅ Merged **Merged:** 10/31/2023 **Merged by:** [@eyurtsev](https://github.com/eyurtsev) **Base:** `main` ← **Head:** `eugene/add_input_validation_streaming` --- ### 📝 Commits (6) - [`82b8a3b`](https://github.com/langchain-ai/langserve/commit/82b8a3b3a7921d43c55475b6bcfa4d6fb01c94af) x - [`2a6dd0f`](https://github.com/langchain-ai/langserve/commit/2a6dd0f370bd36ace073df579d781dfea4a61135) x - [`1dddd88`](https://github.com/langchain-ai/langserve/commit/1dddd880bc7007173bd8226c5e24286b18b2b6e5) x - [`867382a`](https://github.com/langchain-ai/langserve/commit/867382a633e4680cdab1e6b677ccb6c68bef0b2f) x - [`a4ea67f`](https://github.com/langchain-ai/langserve/commit/a4ea67fcae83a748f74be54d483413665bcbd834) x - [`3102368`](https://github.com/langchain-ai/langserve/commit/3102368aea77ccd4957cabbd5ab196dc62bfcb14) Merge branch 'main' into eugene/add_input_validation_streaming ### 📊 Changes **4 files changed** (+663 additions, -314 deletions) <details> <summary>View changed files</summary> 📝 `langserve/server.py` (+329 -194) 📝 `langserve/validation.py` (+31 -0) 📝 `tests/unit_tests/test_server_client.py` (+298 -117) 📝 `tests/unit_tests/test_validation.py` (+5 -3) </details> ### 📄 Description - [x] Add test to verify that client side errors are now raised for streaming endpoints - [x] Add more tests to verify that config and inputs are being validated properly -- or introduce more pydantic models --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 00:20:19 -05:00
yindo closed this issue 2026-02-16 00:20:19 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langserve#369