Bug 11666: remove SQL as an option for MARC framework exports and imports
authorGalen Charlton <gmc@esilibrary.com>
Tue, 4 Feb 2014 23:03:08 +0000 (23:03 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 5 Feb 2014 19:48:27 +0000 (19:48 +0000)
commit94e349ff6ce4a1abb313102decc12429d02dfb4b
tree417159981d382adfaf8c0e8226b58bdb516aed37
parent03d4ed2468bb9ab97b1f7b7d9e29507dc815a8b3
Bug 11666: remove SQL as an option for MARC framework exports and imports

The SQL option for MARC framework imports was subject to a bug whereby
somebody could use it to gain access to arbitrary information in the
database by uploading an SQL file containing unexpected statements.

As it is difficult to securely sanitize SQL, this patch removes the
option to use SQL as an import or export format.

To test:

[1] Verify that SQL no longer appears as an import or export option
    for the MARC frameworks.
[2] Verify that exports and imports in CSV, Excel XML, and ODS formats
    still work.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Works as advertised. The UI doesn't offer exporting/importing in the SQL format.
Crafting the URL to export SQL fallbacks to a spreadsheet format (ODS).

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes all tests and QA script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/ImportExportFramework.pm
admin/import_export_framework.pl
koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt