-
Notifications
You must be signed in to change notification settings - Fork 246
Labels
bugSomething isn't workingSomething isn't workingsecurityImproves securityImproves securitytriageIssues / Features awaiting triageIssues / Features awaiting triage
Milestone
Description
🐞 Bug Summary
User-controlled data in admin UI is not properly escaped.
🧩 Affected Component
-
mcpgateway
- UI (admin panel)
🔁 Steps to Reproduce
- Navigate to admin panel and access "Global Prompts" section
- Click "Add new prompt"
- Enter malicious payload in Template field:
<img src=x onerror="alert('test)">
- Save the prompt
- View the prompt details by clicking the "View" button
- Script executes in browser context
🤔 Expected Behavior
User input should be properly escaped and displayed as text content without executing any embedded scripts.
📓 Logs / Error Output
No server-side errors. Client-side JavaScript alert executes.
🧩 Additional Context (optional)
- Affects admin interface integrity
- Potential for session hijacking or unauthorized admin actions
- Similar escaping issues may exist in other user data display areas
- Admin-only access reduces attack surface but still poses privilege escalation risk
Note: UI can be disabled through feature flag, and listen on local host by default, which significantly reduces exposure in typical deployments where input sanitization is performed by application.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingsecurityImproves securityImproves securitytriageIssues / Features awaiting triageIssues / Features awaiting triage