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)
committerKyle M Hall <kyle@bywatersolutions.com>
Thu, 6 Feb 2014 20:27:21 +0000 (15:27 -0500)
commit50d8aa80fb62afc822a9093bbb2638a687e2487b
tree7f744e7ba9f470a17ee90d5aa3f70077a18055bf
parentc757c2c9c1fe38c8e72f9fdb4f0104c2856f9fde
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>
Conflicts:
koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt
C4/ImportExportFramework.pm
admin/import_export_framework.pl
koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt