Add request specs for caching behavior (#24592)
This commit is contained in:
parent
182bb4595f
commit
2e33a31f80
2 changed files with 707 additions and 0 deletions
|
@ -35,6 +35,26 @@ Devise::Test::ControllerHelpers.module_eval do
|
|||
end
|
||||
end
|
||||
|
||||
module SignedRequestHelpers
|
||||
def get(path, headers: nil, sign_with: nil, **args)
|
||||
return super path, headers: headers, **args if sign_with.nil?
|
||||
|
||||
headers ||= {}
|
||||
headers['Date'] = Time.now.utc.httpdate
|
||||
headers['Host'] = ENV.fetch('LOCAL_DOMAIN')
|
||||
signed_headers = headers.merge('(request-target)' => "get #{path}").slice('(request-target)', 'Host', 'Date')
|
||||
|
||||
key_id = ActivityPub::TagManager.instance.key_uri_for(sign_with)
|
||||
keypair = sign_with.keypair
|
||||
signed_string = signed_headers.map { |key, value| "#{key.downcase}: #{value}" }.join("\n")
|
||||
signature = Base64.strict_encode64(keypair.sign(OpenSSL::Digest.new('SHA256'), signed_string))
|
||||
|
||||
headers['Signature'] = "keyId=\"#{key_id}\",algorithm=\"rsa-sha256\",headers=\"#{signed_headers.keys.join(' ').downcase}\",signature=\"#{signature}\""
|
||||
|
||||
super path, headers: headers, **args
|
||||
end
|
||||
end
|
||||
|
||||
RSpec.configure do |config|
|
||||
config.fixture_path = "#{Rails.root}/spec/fixtures"
|
||||
config.use_transactional_fixtures = true
|
||||
|
@ -46,10 +66,12 @@ RSpec.configure do |config|
|
|||
config.include Devise::Test::ControllerHelpers, type: :helper
|
||||
config.include Devise::Test::ControllerHelpers, type: :view
|
||||
config.include Devise::Test::IntegrationHelpers, type: :feature
|
||||
config.include Devise::Test::IntegrationHelpers, type: :request
|
||||
config.include Paperclip::Shoulda::Matchers
|
||||
config.include ActiveSupport::Testing::TimeHelpers
|
||||
config.include Chewy::Rspec::Helpers
|
||||
config.include Redisable
|
||||
config.include SignedRequestHelpers, type: :request
|
||||
|
||||
config.before :each, type: :feature do
|
||||
https = ENV['LOCAL_HTTPS'] == 'true'
|
||||
|
|
685
spec/requests/cache_spec.rb
Normal file
685
spec/requests/cache_spec.rb
Normal file
|
@ -0,0 +1,685 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
require 'rails_helper'
|
||||
|
||||
module TestEndpoints
|
||||
# Endpoints that do not include authorization-dependent results
|
||||
# and should be cacheable no matter what.
|
||||
ALWAYS_CACHED = %w(
|
||||
/.well-known/host-meta
|
||||
/.well-known/nodeinfo
|
||||
/nodeinfo/2.0
|
||||
/manifest
|
||||
/custom.css
|
||||
/actor
|
||||
/api/v1/instance/extended_description
|
||||
/api/v1/instance/rules
|
||||
/api/v1/instance/peers
|
||||
/api/v1/instance
|
||||
/api/v2/instance
|
||||
).freeze
|
||||
|
||||
# Endpoints that should be cachable when accessed anonymously but have a Vary
|
||||
# on Cookie to prevent logged-in users from getting values from logged-out cache.
|
||||
COOKIE_DEPENDENT_CACHABLE = %w(
|
||||
/
|
||||
/explore
|
||||
/public
|
||||
/about
|
||||
/privacy-policy
|
||||
/directory
|
||||
/@alice
|
||||
/@alice/110224538612341312
|
||||
).freeze
|
||||
|
||||
# Endpoints that should be cachable when accessed anonymously but have a Vary
|
||||
# on Authorization to prevent logged-in users from getting values from logged-out cache.
|
||||
AUTHORIZATION_DEPENDENT_CACHABLE = %w(
|
||||
/api/v1/accounts/lookup?acct=alice
|
||||
/api/v1/statuses/110224538612341312
|
||||
/api/v1/statuses/110224538612341312/context
|
||||
/api/v1/polls/12345
|
||||
/api/v1/trends/statuses
|
||||
/api/v1/directory
|
||||
).freeze
|
||||
|
||||
# Private status that should only be returned with to a valid signature from
|
||||
# a specific user.
|
||||
# Should never be cached.
|
||||
REQUIRE_SIGNATURE = %w(
|
||||
/users/alice/statuses/110224538643211312
|
||||
).freeze
|
||||
|
||||
# Pages only available to logged-in users.
|
||||
# Should never be cached.
|
||||
REQUIRE_LOGIN = %w(
|
||||
/settings/preferences/appearance
|
||||
/settings/profile
|
||||
/settings/featured_tags
|
||||
/settings/export
|
||||
/relationships
|
||||
/filters
|
||||
/statuses_cleanup
|
||||
/auth/edit
|
||||
/oauth/authorized_applications
|
||||
/admin/dashboard
|
||||
).freeze
|
||||
|
||||
# API endpoints only available to logged-in users.
|
||||
# Should never be cached.
|
||||
REQUIRE_TOKEN = %w(
|
||||
/api/v1/announcements
|
||||
/api/v1/timelines/home
|
||||
/api/v1/notifications
|
||||
/api/v1/bookmarks
|
||||
/api/v1/favourites
|
||||
/api/v1/follow_requests
|
||||
/api/v1/conversations
|
||||
/api/v1/statuses/110224538643211312
|
||||
/api/v1/statuses/110224538643211312/context
|
||||
/api/v1/lists
|
||||
/api/v2/filters
|
||||
).freeze
|
||||
|
||||
# Pages that are only shown to logged-out users, and should never get cached
|
||||
# because of CSRF protection.
|
||||
REQUIRE_LOGGED_OUT = %w(
|
||||
/invite/abcdef
|
||||
/auth/sign_in
|
||||
/auth/sign_up
|
||||
/auth/password/new
|
||||
/auth/confirmation/new
|
||||
).freeze
|
||||
|
||||
# Non-exhaustive list of endpoints that feature language-dependent results
|
||||
# and thus need to have a Vary on Accept-Language
|
||||
LANGUAGE_DEPENDENT = %w(
|
||||
/
|
||||
/explore
|
||||
/about
|
||||
/api/v1/trends/statuses
|
||||
).freeze
|
||||
|
||||
module AuthorizedFetch
|
||||
# Endpoints that require a signature with AUTHORIZED_FETCH and LIMITED_FEDERATION_MODE
|
||||
# and thus should not be cached in those modes.
|
||||
REQUIRE_SIGNATURE = %w(
|
||||
/users/alice
|
||||
).freeze
|
||||
end
|
||||
|
||||
module DisabledAnonymousAPI
|
||||
# Endpoints that require a signature with DISALLOW_UNAUTHENTICATED_API_ACCESS
|
||||
# and thus should not be cached in this mode.
|
||||
REQUIRE_TOKEN = %w(
|
||||
/api/v1/custom_emojis
|
||||
).freeze
|
||||
end
|
||||
end
|
||||
|
||||
describe 'Caching behavior' do
|
||||
shared_examples 'cachable response' do
|
||||
it 'does not set cookies' do
|
||||
expect(response.cookies).to be_empty
|
||||
end
|
||||
|
||||
it 'sets public cache control' do
|
||||
# expect(response.cache_control[:max_age]&.to_i).to be_positive
|
||||
expect(response.cache_control[:public]).to be_truthy
|
||||
expect(response.cache_control[:private]).to be_falsy
|
||||
expect(response.cache_control[:no_store]).to be_falsy
|
||||
expect(response.cache_control[:no_cache]).to be_falsy
|
||||
end
|
||||
end
|
||||
|
||||
shared_examples 'non-cacheable response' do
|
||||
it 'sets private cache control' do
|
||||
expect(response.cache_control[:private]).to be_truthy
|
||||
expect(response.cache_control[:no_store]).to be_truthy
|
||||
end
|
||||
end
|
||||
|
||||
shared_examples 'non-cacheable error' do
|
||||
it 'does not return HTTP success' do
|
||||
expect(response).to_not have_http_status(200)
|
||||
end
|
||||
|
||||
it 'does not have cache headers' do
|
||||
expect(response.cache_control[:public]).to be_falsy
|
||||
end
|
||||
end
|
||||
|
||||
shared_examples 'language-dependent' do
|
||||
it 'has a Vary on Accept-Language' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('accept-language')
|
||||
end
|
||||
end
|
||||
|
||||
# Enable CSRF protection like it is in production, as it can cause cookies
|
||||
# to be set and thus mess with cache.
|
||||
around do |example|
|
||||
old = ActionController::Base.allow_forgery_protection
|
||||
ActionController::Base.allow_forgery_protection = true
|
||||
|
||||
example.run
|
||||
|
||||
ActionController::Base.allow_forgery_protection = old
|
||||
end
|
||||
|
||||
let(:alice) { Fabricate(:account, username: 'alice') }
|
||||
let(:user) { Fabricate(:user, role: UserRole.find_by(name: 'Moderator')) }
|
||||
|
||||
before do
|
||||
# rubocop:disable Style/NumericLiterals
|
||||
status = Fabricate(:status, account: alice, id: 110224538612341312)
|
||||
Fabricate(:status, account: alice, id: 110224538643211312, visibility: :private)
|
||||
Fabricate(:invite, code: 'abcdef')
|
||||
Fabricate(:poll, status: status, account: alice, id: 12345)
|
||||
# rubocop:enable Style/NumericLiterals
|
||||
|
||||
user.account.follow!(alice)
|
||||
end
|
||||
|
||||
context 'when anonymously accessed' do
|
||||
TestEndpoints::ALWAYS_CACHED.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::COOKIE_DEPENDENT_CACHABLE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'has a Vary on Cookie' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('cookie')
|
||||
end
|
||||
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::AUTHORIZATION_DEPENDENT_CACHABLE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'has a Vary on Authorization' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('authorization')
|
||||
end
|
||||
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::REQUIRE_LOGGED_OUT.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::REQUIRE_LOGIN + TestEndpoints::REQUIRE_TOKEN).each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
|
||||
describe '/api/v1/instance/domain_blocks' do
|
||||
around do |example|
|
||||
old_setting = Setting.show_domain_blocks
|
||||
Setting.show_domain_blocks = show_domain_blocks
|
||||
|
||||
example.run
|
||||
|
||||
Setting.show_domain_blocks = old_setting
|
||||
end
|
||||
|
||||
before { get '/api/v1/instance/domain_blocks' }
|
||||
|
||||
context 'when set to be publicly-available' do
|
||||
let(:show_domain_blocks) { 'all' }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
end
|
||||
|
||||
context 'when allowed for local users only' do
|
||||
let(:show_domain_blocks) { 'users' }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
|
||||
context 'when disabled' do
|
||||
let(:show_domain_blocks) { 'disabled' }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when logged in' do
|
||||
before do
|
||||
sign_in user, scope: :user
|
||||
|
||||
# Unfortunately, devise's `sign_in` helper causes the `session` to be
|
||||
# loaded in the next request regardless of whether it's actually accessed
|
||||
# by the client code.
|
||||
#
|
||||
# So, we make an extra query to clear issue a session cookie instead.
|
||||
#
|
||||
# A less resource-intensive way to deal with that would be to generate the
|
||||
# session cookie manually, but this seems pretty involved.
|
||||
get '/'
|
||||
end
|
||||
|
||||
TestEndpoints::ALWAYS_CACHED.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::COOKIE_DEPENDENT_CACHABLE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'has a Vary on Cookie' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('cookie')
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::REQUIRE_LOGIN.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::REQUIRE_LOGGED_OUT.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'with an auth token' do
|
||||
let!(:token) { Fabricate(:accessible_access_token, resource_owner_id: user.id, scopes: 'read') }
|
||||
|
||||
TestEndpoints::ALWAYS_CACHED.each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::AUTHORIZATION_DEPENDENT_CACHABLE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'has a Vary on Authorization' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('authorization')
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_LOGGED_OUT + TestEndpoints::REQUIRE_TOKEN).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
describe '/api/v1/instance/domain_blocks' do
|
||||
around do |example|
|
||||
old_setting = Setting.show_domain_blocks
|
||||
Setting.show_domain_blocks = show_domain_blocks
|
||||
|
||||
example.run
|
||||
|
||||
Setting.show_domain_blocks = old_setting
|
||||
end
|
||||
|
||||
before do
|
||||
get '/api/v1/instance/domain_blocks', headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
context 'when set to be publicly-available' do
|
||||
let(:show_domain_blocks) { 'all' }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
end
|
||||
|
||||
context 'when allowed for local users only' do
|
||||
let(:show_domain_blocks) { 'users' }
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
context 'when disabled' do
|
||||
let(:show_domain_blocks) { 'disabled' }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'with a Signature header' do
|
||||
let(:remote_actor) { Fabricate(:account, domain: 'example.org', uri: 'https://example.org/remote', protocol: :activitypub) }
|
||||
let(:dummy_signature) { 'dummy-signature' }
|
||||
|
||||
before do
|
||||
remote_actor.follow!(alice)
|
||||
end
|
||||
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::REQUIRE_SIGNATURE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when enabling AUTHORIZED_FETCH mode' do
|
||||
around do |example|
|
||||
ClimateControl.modify AUTHORIZED_FETCH: 'true' do
|
||||
example.run
|
||||
end
|
||||
end
|
||||
|
||||
context 'when not providing a Signature' do
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::AuthorizedFetch::REQUIRE_SIGNATURE).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when providing a Signature' do
|
||||
let(:remote_actor) { Fabricate(:account, domain: 'example.org', uri: 'https://example.org/remote', protocol: :activitypub) }
|
||||
let(:dummy_signature) { 'dummy-signature' }
|
||||
|
||||
before do
|
||||
remote_actor.follow!(alice)
|
||||
end
|
||||
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::AuthorizedFetch::REQUIRE_SIGNATURE).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when enabling LIMITED_FEDERATION_MODE mode' do
|
||||
around do |example|
|
||||
ClimateControl.modify LIMITED_FEDERATION_MODE: 'true' do
|
||||
old_whitelist_mode = Rails.configuration.x.whitelist_mode
|
||||
Rails.configuration.x.whitelist_mode = true
|
||||
|
||||
example.run
|
||||
|
||||
Rails.configuration.x.whitelist_mode = old_whitelist_mode
|
||||
end
|
||||
end
|
||||
|
||||
context 'when not providing a Signature' do
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::AuthorizedFetch::REQUIRE_SIGNATURE).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when providing a Signature from an allowed domain' do
|
||||
let(:remote_actor) { Fabricate(:account, domain: 'example.org', uri: 'https://example.org/remote', protocol: :activitypub) }
|
||||
let(:dummy_signature) { 'dummy-signature' }
|
||||
|
||||
before do
|
||||
DomainAllow.create!(domain: remote_actor.domain)
|
||||
remote_actor.follow!(alice)
|
||||
end
|
||||
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::AuthorizedFetch::REQUIRE_SIGNATURE).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when providing a Signature from a non-allowed domain' do
|
||||
let(:remote_actor) { Fabricate(:account, domain: 'example.org', uri: 'https://example.org/remote', protocol: :activitypub) }
|
||||
let(:dummy_signature) { 'dummy-signature' }
|
||||
|
||||
describe '/actor' do
|
||||
before do
|
||||
get '/actor', sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_SIGNATURE + TestEndpoints::AuthorizedFetch::REQUIRE_SIGNATURE).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, sign_with: remote_actor, headers: { 'Accept' => 'application/activity+json' }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'when enabling DISALLOW_UNAUTHENTICATED_API_ACCESS' do
|
||||
around do |example|
|
||||
ClimateControl.modify DISALLOW_UNAUTHENTICATED_API_ACCESS: 'true' do
|
||||
example.run
|
||||
end
|
||||
end
|
||||
|
||||
context 'when anonymously accessed' do
|
||||
TestEndpoints::ALWAYS_CACHED.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::REQUIRE_LOGGED_OUT.each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_TOKEN + TestEndpoints::AUTHORIZATION_DEPENDENT_CACHABLE + TestEndpoints::DisabledAnonymousAPI::REQUIRE_TOKEN).each do |endpoint|
|
||||
describe endpoint do
|
||||
before { get endpoint }
|
||||
|
||||
it_behaves_like 'non-cacheable error'
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'with an auth token' do
|
||||
let!(:token) { Fabricate(:accessible_access_token, resource_owner_id: user.id, scopes: 'read') }
|
||||
|
||||
TestEndpoints::ALWAYS_CACHED.each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'cachable response'
|
||||
it_behaves_like 'language-dependent' if TestEndpoints::LANGUAGE_DEPENDENT.include?(endpoint)
|
||||
end
|
||||
end
|
||||
|
||||
TestEndpoints::AUTHORIZATION_DEPENDENT_CACHABLE.each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'has a Vary on Authorization' do
|
||||
expect(response.headers['Vary']&.split(',')&.map { |x| x.strip.downcase }).to include('authorization')
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
(TestEndpoints::REQUIRE_LOGGED_OUT + TestEndpoints::REQUIRE_TOKEN + TestEndpoints::DisabledAnonymousAPI::REQUIRE_TOKEN).each do |endpoint|
|
||||
describe endpoint do
|
||||
before do
|
||||
get endpoint, headers: { 'Authorization' => "Bearer #{token.token}" }
|
||||
end
|
||||
|
||||
it_behaves_like 'non-cacheable response'
|
||||
|
||||
it 'returns HTTP success' do
|
||||
expect(response).to have_http_status(200)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
Loading…
Reference in a new issue