Skip to content

fix: always exit propagate_attributes context manager to prevent session leak #302

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

Closed
sheepdestroyer wants to merge 8 commits into from
Closed

fix: always exit propagate_attributes context manager to prevent session leak #302

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
5d06463
fix: langfuse v4 SDK compatibility + parent observation lifecycle
Jul 13, 2026
655844d
fix: address all PR #296 review comments — Langfuse trace lifecycle h…
Jul 13, 2026
f07ead8
Merge branch 'master' into pr-285
sheepdestroyer Jul 13, 2026
505d2ed
fix: LLAMA_SERVER_URL via config placeholder + canonical URL (was har…
Jul 13, 2026
d763ddb
fix: handle GeneratorExit in streaming generators + use propagate_att…
Jul 13, 2026
e832600
fix: ensure _prop_ctx context manager is always exited (Gemini Code A…
Jul 13, 2026
07d91c8
feat: add Langfuse session propagation test to canonical verification
Jul 13, 2026
a823051
fix: address Gemini review — contextvar task isolation, excessive flu…
Jul 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .env.dev
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,4 @@ ROUTER_IMAGE="localhost/llm-routing-dev:latest"

# Shared infrastructure (same llama classifier across dev & prod)
LLAMA_CLASSIFIER_URL="https://x570.vendeuvre.lan/llm-routing/llama/v1"
LLAMA_SERVER_URL="https://x570.vendeuvre.lan/llm-routing/llama"
2 changes: 2 additions & 0 deletions router/config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ router:
api_key: "os.environ/ROUTER_API_KEY"
model: "qwen-4b-routing"

llama_server_url: "os.environ/LLAMA_SERVER_URL"

classification_rules:
system_prompt: |
Classify the coding task complexity. Output ONLY the tier name.
Expand Down
Loading