adixon / ca.civicrm.logviewer

CiviCRM Log Viewer
Other
9 stars 11 forks source link

sanitize output to fix XSS vulnerability #13

Closed GreysonStalcup closed 1 year ago

GreysonStalcup commented 1 year ago

sanitize output to fix XSS vulnerability - Specifically issue #11

POC - Example of injecting window.location to navigate the administrator from the admin console - Possibilities are endless for this. In this case, I specified my GitHub profile.

(Ignore the errors - Local dev site used for example). xss_vulnerability

image

Fix: Simple addition of htmlentities on the $entry.