[PR #361] [MERGED] Add ability to pass path dependencies through add_routes #516

Closed
opened 2026-02-16 01:15:13 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langserve/pull/361
Author: @eyurtsev
Created: 1/3/2024
Status: Merged
Merged: 1/3/2024
Merged by: @eyurtsev

Base: mainHead: eugene/expose_dependencies


📝 Commits (6)

📊 Changes

3 files changed (+210 additions, -24 deletions)

View changed files

examples/auth/path_dependencies/server.py (+50 -0)
📝 langserve/server.py (+63 -15)
📝 tests/unit_tests/test_server_client.py (+97 -9)

📄 Description

This PR adds the ability to specify path dependencies through add_routes.

This is useful whenever the dependencies need to be solved, but their return
value is not important; e.g., this could be a simple way to do authentication,
assuming all authenticated users have the same authorization privilegies.

https://github.com/langchain-ai/langserve/issues/354


🔄 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/361 **Author:** [@eyurtsev](https://github.com/eyurtsev) **Created:** 1/3/2024 **Status:** ✅ Merged **Merged:** 1/3/2024 **Merged by:** [@eyurtsev](https://github.com/eyurtsev) **Base:** `main` ← **Head:** `eugene/expose_dependencies` --- ### 📝 Commits (6) - [`4aacd6c`](https://github.com/langchain-ai/langserve/commit/4aacd6ca5c3de3873eef6e1741f0a22143ff4915) x - [`aca37a3`](https://github.com/langchain-ai/langserve/commit/aca37a397ad39a00fc3dba4f290d1dd637bb357e) x - [`3620a60`](https://github.com/langchain-ai/langserve/commit/3620a60f69db1708a4467122a6c845a05031f4f4) make test more robust - [`74f5a10`](https://github.com/langchain-ai/langserve/commit/74f5a1054c64daf42b020f289cde269c041ae0ad) x - [`bf9e65b`](https://github.com/langchain-ai/langserve/commit/bf9e65b8b44231f32f642223f17a28034b87a6e2) x - [`5e6e60c`](https://github.com/langchain-ai/langserve/commit/5e6e60cf86dfe663e44a5de19822ba212b63b365) x ### 📊 Changes **3 files changed** (+210 additions, -24 deletions) <details> <summary>View changed files</summary> ➕ `examples/auth/path_dependencies/server.py` (+50 -0) 📝 `langserve/server.py` (+63 -15) 📝 `tests/unit_tests/test_server_client.py` (+97 -9) </details> ### 📄 Description This PR adds the ability to specify path dependencies through add_routes. This is useful whenever the dependencies need to be solved, but their return value is not important; e.g., this could be a simple way to do authentication, assuming all authenticated users have the same authorization privilegies. https://github.com/langchain-ai/langserve/issues/354 --- <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 01:15:13 -05:00
yindo closed this issue 2026-02-16 01:15:13 -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#516