diff --git a/lib/middleware/anonymous_cache.rb b/lib/middleware/anonymous_cache.rb index 8d605ae537c..cb06d8793f0 100644 --- a/lib/middleware/anonymous_cache.rb +++ b/lib/middleware/anonymous_cache.rb @@ -315,7 +315,7 @@ module Middleware if PAYLOAD_INVALID_REQUEST_METHODS.include?(env[Rack::REQUEST_METHOD]) && env[Rack::RACK_INPUT].size > 0 - return [413, {}, []] + return [413, { "Cache-Control" => "private, max-age=0, must-revalidate" }, []] end helper = Helper.new(env) diff --git a/lib/middleware/discourse_public_exceptions.rb b/lib/middleware/discourse_public_exceptions.rb index 8b9cbc827df..fcc2b5ed268 100644 --- a/lib/middleware/discourse_public_exceptions.rb +++ b/lib/middleware/discourse_public_exceptions.rb @@ -35,7 +35,7 @@ module Middleware begin request.format rescue Mime::Type::InvalidMimeType - return [400, {}, ["Invalid MIME type"]] + return [400, { "Cache-Control" => "private, max-age=0, must-revalidate" }, ["Invalid MIME type"]] end if ApplicationController.rescue_with_handler(exception, object: fake_controller) diff --git a/spec/components/middleware/anonymous_cache_spec.rb b/spec/components/middleware/anonymous_cache_spec.rb index e0b23c8a54f..9cf0e058663 100644 --- a/spec/components/middleware/anonymous_cache_spec.rb +++ b/spec/components/middleware/anonymous_cache_spec.rb @@ -240,11 +240,12 @@ describe Middleware::AnonymousCache do context 'invalid request payload' do it 'returns 413 for GET request with payload' do - status, _, _ = middleware.call(env.tap do |environment| + status, headers, _ = middleware.call(env.tap do |environment| environment[Rack::RACK_INPUT].write("test") end) expect(status).to eq(413) + expect(headers["Cache-Control"]).to eq("private, max-age=0, must-revalidate") end end