-
Type:
Bug
-
Status: Waiting for Dev
-
Priority:
Should
-
Resolution: Unresolved
-
Affects Version/s: None
-
Fix Version/s: Reference Application 2.10.0
-
Component/s: None
-
Labels:
-
Complexity:Low
There is several pages in the RA that are vulnerable to XSS attacks, a vulnerability can be reproduced when creating a patient and the JS is specified in the surname and address fields, see https://talk.openmrs.org/t/xss-vulnerability-in-openmrs-2-x-ui/698 and several other pages under 'Configure Metadata' and 'System Administration' apps.
There is more vulnerabilities mentioned at http://packetstormsecurity.com/files/128748
Would be worth it to investigate other vulnerable areas in the UI
Acceptance Criteria
- The page https://wiki.openmrs.org/x/h5UvAg should have a section documenting how to escape user-entered content using UiUtils.escape* methods, and examples
- The specific vulnerability mentioned in https://talk.openmrs.org/t/xss-vulnerability-in-openmrs-2-x-ui/698 is fixed
- A new ticket exists to go through the whole 2.x UI and clean up XSS vulnerabilities
Process Notes
We should create subtasks for (groups of) vulnerabilities, and let people fix them in parallel.
1.
|
Patient names aren't sanitized, enabling malicious activity |
|
Accepted | Daniel Kayiwa |
2.
|
XSS on phone number field |
|
Accepted | Daniel Kayiwa |
3.
|
Stored XSS in Patient Address |
|
Accepted | Daniel Kayiwa |
4.
|
Stored XSS in name and description fields in reports |
|
Accepted | Daniel Kayiwa |
5.
|
Stored XSS in appointment types |
|
Accepted | Daniel Kayiwa |
6.
|
Stored XSS in locations |
|
Accepted | Daniel Kayiwa |
7.
|
Stored XSS in telephone number field |
|
Waiting for Showcase | Unassigned |
8.
|
Stored XSS in Diagnoses section of patient.page |
|
Accepted | Daniel Kayiwa |
9.
|
Reflected XSS in returnUrl parameter |
|
Waiting for Test | Isaac Sears [X] (Inactive) |
10.
|
escapeJs vulnerable to XSS |
|
Waiting for Analysis | Unassigned |
11.
|
XSS attack in toast messages after patient registrationion |
|
Accepted | Wyclif Luyima |
12.
|
XSS attack in toast messages after patient visit |
|
Accepted | Wyclif Luyima |
