Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: support qdrant url path prefixes#137

Open
zxyasfas wants to merge 1 commit into
qdrant:masterfrom
zxyasfas:fix/qdrant-url-prefix
Open

fix: support qdrant url path prefixes#137
zxyasfas wants to merge 1 commit into
qdrant:masterfrom
zxyasfas:fix/qdrant-url-prefix

Conversation

@zxyasfas
Copy link
Copy Markdown

@zxyasfas zxyasfas commented May 6, 2026

Summary

Fixes QDRANT_URL values that include a reverse-proxy path prefix, such as https://example.com/qdrant.

Problem

When the server runs behind Kubernetes ingress, Nginx, Traefik, or another reverse proxy, Qdrant may be exposed under a path prefix and standard HTTP(S) ports. Passing that URL through as a plain client location keeps the default Qdrant port and can make the client connect to the wrong endpoint.

Changes

  • Split prefixed QDRANT_URL values into a base url and Qdrant client prefix.
  • Use standard ports for prefixed https and http URLs when no explicit port is present.
  • Preserve existing behavior for regular Qdrant URLs, explicit ports, :memory:, and local-path mode.
  • Added tests for prefixed URLs, explicit ports, and connector option forwarding.

Fixes #135.

Validation

  • .venv\Scripts\python.exe -m pytest tests\test_qdrant_client_options.py
  • .venv\Scripts\ruff.exe check src\mcp_server_qdrant\qdrant.py tests\test_qdrant_client_options.py
  • .venv\Scripts\python.exe -m py_compile src\mcp_server_qdrant\qdrant.py tests\test_qdrant_client_options.py

@zxyasfas
Copy link
Copy Markdown
Author

zxyasfas commented May 8, 2026

Updated the validation section after rerunning the targeted checks locally. The focused test covers prefixed HTTPS/HTTP URLs, explicit ports, non-HTTP locations, and connector option forwarding; ruff and py_compile also pass for the touched files.

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.

Support QDRANT_URL with path prefix (reverse proxy deployments)

1 participant