4bec613897
This is a big one, so let me enumerate: Accounts as well as stream entry pages now contain Link headers that reference the Atom feed and Webfinger URL for the former and Atom entry for the latter. So you only need to HEAD those resources to get that information, no need to download and parse HTML <link>s. ProcessFeedService will now queue ThreadResolveWorker for each remote status that it cannot find otherwise. Furthermore, entries are now processed in reverse order (from bottom to top) in case a newer entry references a chronologically previous one. ThreadResolveWorker uses FetchRemoteStatusService to obtain a status and attach the child status it was queued for to it. FetchRemoteStatusService looks up the URL, first with a HEAD, tests if it's an Atom feed, in which case it processes it directly. Next for Link headers to the Atom feed, in which case that is fetched and processed. Lastly if it's HTML, it is checked for <link>s to the Atom feed, and if such is found, that is fetched and processed. The account for the status is derived from author/name attribute in the XML and the hostname in the URL (domain). FollowRemoteAccountService and ProcessFeedService are used. This means that potentially threads are resolved recursively until a dead-end is encountered, however it is performed asynchronously over background jobs, so it should be ok.
51 lines
2.0 KiB
Ruby
51 lines
2.0 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe Api::SubscriptionsController, type: :controller do
|
|
render_views
|
|
|
|
let(:account) { Fabricate(:account, username: 'gargron', domain: 'quitter.no', remote_url: 'topic_url', secret: 'abc') }
|
|
|
|
describe 'GET #show' do
|
|
before do
|
|
get :show, params: { :id => account.id, 'hub.topic' => 'topic_url', 'hub.challenge' => '456', 'hub.lease_seconds' => "#{86400 * 30}" }
|
|
end
|
|
|
|
it 'returns http success' do
|
|
expect(response).to have_http_status(:success)
|
|
end
|
|
|
|
it 'echoes back the challenge' do
|
|
expect(response.body).to match '456'
|
|
end
|
|
end
|
|
|
|
describe 'POST #update' do
|
|
let(:feed) { File.read(File.join(Rails.root, 'spec', 'fixtures', 'push', 'feed.atom')) }
|
|
|
|
before do
|
|
stub_request(:get, "https://quitter.no/avatar/7477-300-20160211190340.png").to_return(request_fixture('avatar.txt'))
|
|
stub_request(:head, "https://quitter.no/notice/1269244").to_return(status: 404)
|
|
stub_request(:head, "https://quitter.no/notice/1265331").to_return(status: 404)
|
|
stub_request(:head, "https://community.highlandarrow.com/notice/54411").to_return(status: 404)
|
|
stub_request(:head, "https://community.highlandarrow.com/notice/53857").to_return(status: 404)
|
|
stub_request(:head, "https://community.highlandarrow.com/notice/51852").to_return(status: 404)
|
|
stub_request(:head, "https://social.umeahackerspace.se/notice/424348").to_return(status: 404)
|
|
stub_request(:head, "https://community.highlandarrow.com/notice/50467").to_return(status: 404)
|
|
stub_request(:head, "https://quitter.no/notice/1243309").to_return(status: 404)
|
|
|
|
request.env['HTTP_X_HUB_SIGNATURE'] = "sha1=#{OpenSSL::HMAC.hexdigest('sha1', 'abc', feed)}"
|
|
request.env['RAW_POST_DATA'] = feed
|
|
|
|
post :update, params: { id: account.id }
|
|
end
|
|
|
|
it 'returns http created' do
|
|
expect(response).to have_http_status(:created)
|
|
end
|
|
|
|
it 'creates statuses for feed' do
|
|
expect(account.statuses.count).to_not eq 0
|
|
end
|
|
end
|
|
end
|