Skip to content

Fix Windows SQLite path parsing in UI server and import#25

Merged
erans merged 1 commit intoerans:mainfrom
njt:fix/windows-sqlite-path
Mar 11, 2026
Merged

Fix Windows SQLite path parsing in UI server and import#25
erans merged 1 commit intoerans:mainfrom
njt:fix/windows-sqlite-path

Conversation

@njt
Copy link
Copy Markdown

@njt njt commented Dec 25, 2025

Use SqliteConnectOptions::new().filename() instead of URL-based connection strings. Windows paths like C:\Users... contain backslashes that are misinterpreted by the SQLite URL parser, causing errors like: "unknown query parameter \C:\Users..."

This matches the pattern already used in sqlite_writer.rs which works correctly on all platforms.

🤖 Generated with Claude Code for my sins.

Use SqliteConnectOptions::new().filename() instead of URL-based
connection strings. Windows paths like C:\Users\... contain backslashes
that are misinterpreted by the SQLite URL parser, causing errors like:
"unknown query parameter \C:\Users\..."

This matches the pattern already used in sqlite_writer.rs which works
correctly on all platforms.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@erans erans merged commit d6e0a82 into erans:main Mar 11, 2026
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