FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add thread_id to middlewares by mateusz834 · Pull Request #755 · splunk/splunk-sdk-python · GitHub

Repository navigation

Add thread_id to middlewares - #755

Merged
mateusz834 merged 1 commit into
developfrom
dev/MP/thread-id-middlewares
May 5, 2026
Merged

mateusz834 merged 1 commit into
developfrom
dev/MP/thread-id-middlewares

Conversation

Copy link
Copy Markdown
Member

Additionally make sure that subagents get an unique thread_id when no conversation store is being used.

And enforce that thread_id cannot be an empty string, as that is clearly a bug.

mateusz834 force-pushed the dev/MP/thread-id-middlewares branch 2 times, most recently from 7259546 to 338f820 Compare April 29, 2026 13:12
mateusz834 marked this pull request as ready for review April 29, 2026 13:13
mateusz834 force-pushed the dev/MP/thread-id-middlewares branch 2 times, most recently from 0fd896e to 29bea8d Compare April 29, 2026 13:15
Comment thread tests/integration/ai/test_agent.py Resolved
Comment thread splunklib/ai/engines/langchain.py Resolved
Additionally make sure that subagents get an unique thread_id
when no conversation store is being used.

And enforce that thread_id cannot be an empty string, as that
is clearly a bug.
mateusz834 force-pushed the dev/MP/thread-id-middlewares branch from 29bea8d to 11d5bcb Compare May 4, 2026 10:14
mateusz834 merged commit 7274e35 into develop May 5, 2026
5 of 7 checks passed
mateusz834 deleted the dev/MP/thread-id-middlewares branch May 5, 2026 08:38
mateusz834 added a commit that referenced this pull request May 5, 2026
I have not rebased #755, but after merge it semantically conflicted with
#758.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL