Bug 31219: Prevent JS injection in patron extended attributes
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 25 Jul 2022 07:23:25 +0000 (09:23 +0200)
committerLucas Gass <lucas@bywatersolutions.com>
Mon, 26 Sep 2022 14:08:41 +0000 (14:08 +0000)
commitddaa78bdab360f41b9e3672de7f50cd3053b9116
tree2ce5fa28fda85e93175b221b7d806a19aaef4cee
parentc3c006713bd8bc457b73dbcd3423a2506369bf9e
Bug 31219: Prevent JS injection in patron extended attributes

We are sanitizing other attributes but "extended patron attributes".

Test plan:
Make a patron attribute editable at the OPAC
Edit an existing patron, or register a new one
Use a script tag in the new value ("<script>alert("booh!")</script>" for
instance)
With this patch the value is remove if containing an HTML tag that is
not br b i em big small strong (see C4::Scrubber)

Signed-off-by: Mark Hofstetter <koha@trust-box.at>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
opac/opac-memberentry.pl