Browse Source

fix CSP / X-Frame-Options for media embeds (#9558)

jomo 5 years ago
parent
commit
2c1a6f746f
1 changed files with 5 additions and 0 deletions
  1. 5 0
      app/controllers/media_controller.rb

+ 5 - 0
app/controllers/media_controller.rb

@@ -6,12 +6,17 @@ class MediaController < ApplicationController
   before_action :set_media_attachment
   before_action :verify_permitted_status!
 
+  content_security_policy only: :player do |p|
+    p.frame_ancestors(false)
+  end
+
   def show
     redirect_to @media_attachment.file.url(:original)
   end
 
   def player
     @body_classes = 'player'
+    response.headers['X-Frame-Options'] = 'ALLOWALL'
     raise ActiveRecord::RecordNotFound unless @media_attachment.video? || @media_attachment.gifv?
   end