me/glitchier-soc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix caching headers in ActivityPub endpoints (#11331)

Browse source 
* Fix reverse-proxy caching in public fetch mode

* Fix caching in ActivityPub-specific controllers
This commit is contained in:
ThibG 2019-07-17 00:00:39 +02:00 committed by Eugen Rochko
parent ab6af77728
commit a4b1083795
5 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
app/controllers/activitypub/base_controller.rb Normal file
View file

@ -0,0 +1,9 @@
# frozen_string_literal: true
class ActivityPub::BaseController < Api::BaseController
private
def set_cache_headers
response.headers['Vary'] = 'Signature' if authorized_fetch_mode?
end
end

2
app/controllers/activitypub/collections_controller.rb
View file

@ -1,6 +1,6 @@
# frozen_string_literal: true
class ActivityPub::CollectionsController < Api::BaseController
class ActivityPub::CollectionsController < ActivityPub::BaseController
include SignatureVerification
include AccountOwnedConcern

2
app/controllers/activitypub/outboxes_controller.rb
View file

@ -1,6 +1,6 @@
# frozen_string_literal: true
class ActivityPub::OutboxesController < Api::BaseController
class ActivityPub::OutboxesController < ActivityPub::BaseController
LIMIT = 20
include SignatureVerification

2
app/controllers/activitypub/replies_controller.rb
View file

@ -1,6 +1,6 @@
# frozen_string_literal: true
class ActivityPub::RepliesController < Api::BaseController
class ActivityPub::RepliesController < ActivityPub::BaseController
include SignatureAuthentication
include Authorization
include AccountOwnedConcern

2
app/controllers/application_controller.rb
View file

@ -160,6 +160,6 @@ class ApplicationController < ActionController::Base
end
def set_cache_headers
response.headers['Vary'] = 'Accept, Signature'
response.headers['Vary'] = public_fetch_mode? ? 'Accept' : 'Accept, Signature'
end
end