Skip to content

Fix Python vector search authentication#2

Draft
SWORDIntel wants to merge 1 commit into
mainfrom
agent/fix-python-vector-search-auth
Draft

Fix Python vector search authentication#2
SWORDIntel wants to merge 1 commit into
mainfrom
agent/fix-python-vector-search-auth

Conversation

@SWORDIntel

Copy link
Copy Markdown
Owner

Summary

  • align the Python ctypes vector query layout with the native qihse_vector_query_t ABI
  • initialize and retain the process-local SDK user required by authorization-aware search
  • attach that user to every Python vector query
  • add a regression test covering authenticated native vector search

Root cause

The native query struct gained a required user field, but the Python ctypes definition was not updated. Native search read beyond the shorter Python struct, producing EACCES or invalid pointers in isolated worker processes.

Validation

  • PYTHONPATH=python python3 -m pytest -q python/tests/test_vector_search_auth.py python/tests/test_recall.py
  • FRAMEWERX QIHSE wrapper suite: 23 passed
  • git diff --check

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.

1 participant