Revert "RawController respond with uncached headers"

This reverts commit 494c56ef.

app/controllers/projects/raw_controller.rb

@@ -12,7 +12,6 @@ class Projects::RawController < Projects::ApplicationController
   before_action :authorize_download_code!
   before_action :show_rate_limit, only: [:show], unless: :external_storage_request?
   before_action :assign_ref_vars
-  before_action :no_cache_headers, only: [:show]
   before_action :redirect_to_external_storage, only: :show, if: :static_objects_external_storage_enabled?
 
   feature_category :source_code_management

spec/controllers/projects/raw_controller_spec.rb

@@ -33,11 +33,6 @@ RSpec.describe Projects::RawController do
 
       it_behaves_like 'project cache control headers'
       it_behaves_like 'content disposition headers'
-      it_behaves_like 'uncached response' do
-        before do
-          subject
-        end
-      end
     end
 
     context 'image header' do

spec/requests/api/files_spec.rb

@@ -537,13 +537,16 @@ RSpec.describe API::Files do
         expect(response).to have_gitlab_http_status(:ok)
       end
 
-      it_behaves_like 'uncached response' do
-        before do
-          url = route('.gitignore') + "/raw"
-          expect(Gitlab::Workhorse).to receive(:send_git_blob)
+      it 'sets no-cache headers' do
+        url = route('.gitignore') + "/raw"
+        expect(Gitlab::Workhorse).to receive(:send_git_blob)
 
-          get api(url, current_user), params: params
-        end
+        get api(url, current_user), params: params
+
+        expect(response.headers["Cache-Control"]).to include("no-store")
+        expect(response.headers["Cache-Control"]).to include("no-cache")
+        expect(response.headers["Pragma"]).to eq("no-cache")
+        expect(response.headers["Expires"]).to eq("Fri, 01 Jan 1990 00:00:00 GMT")
       end
 
       context 'when mandatory params are not given' do

spec/support/shared_examples/cached_response_shared_examples.rb

-# frozen_string_literal: true
-#
-# Negates lib/gitlab/no_cache_headers.rb
-#
-
-RSpec.shared_examples 'cached response' do
-  it 'defines a cached header response' do
-    expect(response.headers["Cache-Control"]).not_to include("no-store", "no-cache")
-    expect(response.headers["Pragma"]).not_to eq("no-cache")
-    expect(response.headers["Expires"]).not_to eq("Fri, 01 Jan 1990 00:00:00 GMT")
-  end
-end