Skip to content

fix: resolve dashboard 500 errors by injecting Qdrant point IDs into log retrieval#147

Open
paramsavla06 wants to merge 1 commit into
Dharanish-AM:mainfrom
paramsavla06:fix-inject-qdrant
Open

fix: resolve dashboard 500 errors by injecting Qdrant point IDs into log retrieval#147
paramsavla06 wants to merge 1 commit into
Dharanish-AM:mainfrom
paramsavla06:fix-inject-qdrant

Conversation

@paramsavla06

Copy link
Copy Markdown

Summary

Fixes the /logs and /search endpoints returning 500 Internal Server Error
when the dashboard attempts to fetch logs from Qdrant.

Problem

  • LogDTO requires an id field, but Qdrant stores point IDs at the record
    root (Record.id), not inside the custom payload dict. This caused a
    Pydantic ValidationError on every retrieval call.
  • The default CORS origins list was missing localhost:5174, blocking
    frontend dev servers that fall back to that port.

Changes

  • services/log_service.py: Updated get_logs() and semantic_search()
    to copy r.id into the returned payload dictionaries.
  • app_factory.py: Added http://localhost:5174 to the default
    allowed_origins fallback list.

Testing

  • Verified GET /logs?limit=50 returns 200 OK with properly serialized
    LogDTO objects including id fields.
  • Confirmed dashboard UI connects without CORS errors.

@github-actions github-actions Bot added backend Backend application changes or issues frontend Frontend application changes or issues infrastructure CI, CD, Docker, or repository automation work gssoc26 GSSoC 2026 Contribution gssoc:approved Approved for GSSoC 2026 labels Jun 11, 2026
@github-actions

Copy link
Copy Markdown

⚠️ Merge Conflict Detected! This PR cannot be merged automatically because it conflicts with the main branch.

@🎨 Contributor: Please update your branch locally, resolve the conflicts, and push the updates. The pipeline has skipped this PR for now and moved on! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend application changes or issues frontend Frontend application changes or issues gssoc:approved Approved for GSSoC 2026 gssoc26 GSSoC 2026 Contribution infrastructure CI, CD, Docker, or repository automation work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant