Skip to content

fix: remove hardcoded /tmp in create_client()#6

Open
ton77v wants to merge 1 commit intomainfrom
fix/tempdir-windows-compat
Open

fix: remove hardcoded /tmp in create_client()#6
ton77v wants to merge 1 commit intomainfrom
fix/tempdir-windows-compat

Conversation

@ton77v
Copy link
Copy Markdown

@ton77v ton77v commented Mar 16, 2026

Summary

  • create_client() hardcodes dir="/tmp" in tempfile.mkdtemp() — crashes on Windows
  • Fix: drop dir arg, let Python use platform-default temp directory

Fixes #5

🦀 sent by Claude Code

Fixes #5

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ton77v ton77v requested a review from sscarduzio March 16, 2026 03:30
@sscarduzio
Copy link
Copy Markdown
Collaborator

Ok but why using this? We have deltaglider proxy

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.

create_client() hardcodes /tmp — breaks on Windows

2 participants