Commit e2a56af9 authored by Rémy Coutable's avatar Rémy Coutable 🏔

Merge branch 'sh-fix-cross-site-forgery-errors' into 'master'

Return a blank JSON response for a missing .js file to prevent Rails CSRF errors

Closes #40771

See merge request gitlab-org/gitlab-ce!16664
parents 47f6dbdf 79a829a0
Pipeline #16554804 failed with stages
in 44 minutes and 33 seconds
......@@ -147,6 +147,8 @@ class ApplicationController < ActionController::Base
format.html do
render file: Rails.root.join("public", "404"), layout: false, status: "404"
# Prevent the Rails CSRF protector from thinking a missing .js file is a JavaScript file
format.js { render json: '', status: :not_found, content_type: 'application/json' }
format.any { head :not_found }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment