[freeside-commits] branch master updated. dfa43146bb9c472322b82c18a912f44401a76694

Ivan ivan at 420.am
Fri Jan 15 15:46:33 PST 2016


The branch, master has been updated
       via  dfa43146bb9c472322b82c18a912f44401a76694 (commit)
      from  e74ab331f1d713d9baf4daa12834fe7d9028520d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit dfa43146bb9c472322b82c18a912f44401a76694
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Fri Jan 15 15:46:32 2016 -0800

    fix self-service fallout from clickjacking fix, RT#39881

diff --git a/FS/FS/Mason/Request.pm b/FS/FS/Mason/Request.pm
index b33efcc..0d9c6d4 100644
--- a/FS/FS/Mason/Request.pm
+++ b/FS/FS/Mason/Request.pm
@@ -65,9 +65,11 @@ sub freeside_setup {
             if fileno(STDOUT) != 1;
     }
 
-    FS::Trace->log('    adding headers');
-    #frame-ancestors not supported by all the major browsers yet
-    $HTML::Mason::Commands::r->header_out( 'X-Frame-Options', 'SAMEORIGIN' );
+    if ( $HTML::Mason::Commands::r ) {
+      FS::Trace->log('    adding headers');
+      #frame-ancestors not supported by all the major browsers yet
+      $HTML::Mason::Commands::r->header_out( 'X-Frame-Options', 'SAMEORIGIN' );
+    }
 
     if ( $filename =~ qr(/REST/\d+\.\d+/NoAuth/) ) {
 

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/Mason/Request.pm |    8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)




More information about the freeside-commits mailing list