Bug 15111: Change X-Frame-Options with SAMEORIGIN
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 13 Nov 2015 08:19:57 +0000 (08:19 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Mon, 14 Mar 2016 16:30:08 +0000 (16:30 +0000)
There are some places where frames are used, the greybox JS plugin for
instance.

We need either to allow them from Koha or replace this plugin.
The easier for now is to switch the value from DENY with SAMEORIGIN.

Test plan:
- modify a record in a batch (tools/batch_record_modification.pl)
- click on preview marc
=> With only the previous patch you will get a blank page.
=> With this patch apply, it will work as expected.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/Output.pm

index 0527d65..0a0ae21 100644 (file)
@@ -269,7 +269,7 @@ sub output_with_http_headers {
         charset           => 'UTF-8',
         Pragma            => 'no-cache',
         'Cache-Control'   => $cache_policy,
-        'X-Frame-Options' => 'DENY',
+        'X-Frame-Options' => 'SAMEORIGIN',
     };
     $options->{expires} = 'now' if $extra_options->{force_no_caching};