Skip to content

fix: the application fetches data from an external u... in Datasets.jsx - #439

Merged
generall merged 1 commit into
qdrant:masterfrom
anupamme:fix-repo-qdrant-web-ui-datasets-manifest-validation
Sep 3, 2026
Merged

fix: the application fetches data from an external u... in Datasets.jsx#439
generall merged 1 commit into
qdrant:masterfrom
anupamme:fix-repo-qdrant-web-ui-datasets-manifest-validation

Conversation

@anupamme

@anupamme anupamme commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix high severity security issue in src/pages/Datasets.jsx.

Vulnerability

Field Value
ID V-001
Severity HIGH
Scanner multi_agent_ai
Rule V-001
File src/pages/Datasets.jsx:29
Assessment Likely exploitable

Description: The application fetches data from an external URL (https://snapshots.qdrant.io/manifest-v1.16.0.json) without any integrity validation, cryptographic signature verification, or Subresource Integrity (SRI) checks. The response data is directly used to populate the datasets state and rendered in the UI without sanitization.

Evidence

Exploitation scenario: An attacker who compromises the snapshots.qdrant.io server or performs DNS spoofing/MITM can inject malicious JSON containing XSS payloads.

Scanner confirmation: multi_agent_ai rule V-001 flagged this pattern.

Production code: This file is in the production codebase, not test-only code.

Threat Model Context

This is a web application - XSS and injection vulnerabilities can affect end users.

Changes

  • src/pages/Datasets.jsx

Behavior Preservation

The change is scoped to 1 file on the vulnerable path; it only tightens handling of untrusted input and leaves valid inputs unaffected.


Automated security fix by OrbisAI Security

Automated security fix generated by OrbisAI Security
@generall
generall force-pushed the fix-repo-qdrant-web-ui-datasets-manifest-validation branch from be5047c to 79465ab Compare September 3, 2026 11:03
@generall
generall merged commit 15ab3ed into qdrant:master Sep 3, 2026
1 check passed
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