|Summary:||Data displayed in request list datatable is not sanitized|
|Product:||Koha||Reporter:||Andrew Isherwood <andrew.isherwood>|
|Component:||ILL||Assignee:||Bugs List <koha-bugs>|
|Status:||RESOLVED DUPLICATE||QA Contact:|
|Priority:||P5 - low||CC:||katrin.fischer|
|Change sponsored?:||---||Patch complexity:||---|
|Who signed the patch off:||Text to go in the release notes:|
|Version(s) released in:|
Description Andrew Isherwood 2019-02-04 13:59:10 UTC
Request data displayed in the "Request view" or similar views is sanitized by the html filter in the template. However, the main request list table is generated via Datatables and is not sanitized. This can be demonstrated by entering a malicious string, such as: <script>alert('pwned');</script> Into the request notes field, then displaying the requests table. I've not yet investigated if column data that is generated via render functions is affected, or whether it's just the columns that are passed through by Datatables. Investigate how this has been addressed elsewhere in Koha. Also, possible mitigations can be seen here: https://datatables.net/manual/security
Comment 1 Katrin Fischer 2019-02-04 17:54:48 UTC
Wondering about this one - it currently blocks any progress on the ILL graph. Is it in the right spot of the graph? Should it be in Koha security?
Comment 2 Andrew Isherwood 2019-02-05 17:10:08 UTC
Hi Katrin - I've now moved it to the top of the dependency tree, I'm going to try and get a fix out in the next day or two.