Update hub URL and re-subscribe if hub URL changes
This commit is contained in:
parent
4986c727d9
commit
2cb3dc5e5a
8 changed files with 23 additions and 23 deletions
|
@ -80,8 +80,7 @@ class FollowRemoteAccountService < BaseService
|
||||||
end
|
end
|
||||||
|
|
||||||
def get_profile(xml, account)
|
def get_profile(xml, account)
|
||||||
author = xml.at_xpath('/xmlns:feed/xmlns:author') || xml.at_xpath('/xmlns:feed').at_xpath('./dfrn:owner', dfrn: DFRN_NS)
|
update_remote_profile_service.call(xml.at_xpath('/xmlns:feed'), account)
|
||||||
update_remote_profile_service.call(author, account)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def update_remote_profile_service
|
def update_remote_profile_service
|
||||||
|
|
|
@ -16,7 +16,7 @@ class ProcessFeedService < BaseService
|
||||||
|
|
||||||
def update_author(xml, account)
|
def update_author(xml, account)
|
||||||
return if xml.at_xpath('/xmlns:feed').nil?
|
return if xml.at_xpath('/xmlns:feed').nil?
|
||||||
UpdateRemoteProfileService.new.call(xml.at_xpath('/xmlns:feed/xmlns:author'), account)
|
UpdateRemoteProfileService.new.call(xml.at_xpath('/xmlns:feed'), account, true)
|
||||||
end
|
end
|
||||||
|
|
||||||
def process_entries(xml, account)
|
def process_entries(xml, account)
|
||||||
|
|
|
@ -26,7 +26,7 @@ class ProcessInteractionService < BaseService
|
||||||
end
|
end
|
||||||
|
|
||||||
if salmon.verify(envelope, account.keypair)
|
if salmon.verify(envelope, account.keypair)
|
||||||
update_remote_profile_service.call(xml.at_xpath('/xmlns:entry/xmlns:author'), account)
|
update_remote_profile_service.call(xml.at_xpath('/xmlns:entry'), account, true)
|
||||||
|
|
||||||
case verb(xml)
|
case verb(xml)
|
||||||
when :follow
|
when :follow
|
||||||
|
|
|
@ -2,24 +2,22 @@
|
||||||
|
|
||||||
class UpdateRemoteProfileService < BaseService
|
class UpdateRemoteProfileService < BaseService
|
||||||
POCO_NS = 'http://portablecontacts.net/spec/1.0'
|
POCO_NS = 'http://portablecontacts.net/spec/1.0'
|
||||||
|
DFRN_NS = 'http://purl.org/macgirvin/dfrn/1.0'
|
||||||
|
|
||||||
def call(author_xml, account)
|
def call(xml, account, resubscribe = false)
|
||||||
return if author_xml.nil?
|
author_xml = xml.at_xpath('./xmlns:author') || xml.at_xpath('./dfrn:owner', dfrn: DFRN_NS)
|
||||||
|
hub_link = xml.at_xpath('./xmlns:link[@rel="hub"]')
|
||||||
|
|
||||||
account.display_name = if author_xml.at_xpath('./poco:displayName', poco: POCO_NS).nil?
|
unless author_xml.nil?
|
||||||
account.username
|
account.display_name = author_xml.at_xpath('./poco:displayName', poco: POCO_NS).content unless author_xml.at_xpath('./poco:displayName', poco: POCO_NS).nil?
|
||||||
else
|
account.note = author_xml.at_xpath('./poco:note', poco: POCO_NS).content unless author_xml.at_xpath('./poco:note').nil?
|
||||||
author_xml.at_xpath('./poco:displayName', poco: POCO_NS).content
|
account.avatar_remote_url = author_xml.at_xpath('./xmlns:link[@rel="avatar"]')['href'] unless author_xml.at_xpath('./xmlns:link[@rel="avatar"]').nil? || author_xml.at_xpath('./xmlns:link[@rel="avatar"]')['href'].blank?
|
||||||
end
|
|
||||||
|
|
||||||
unless author_xml.at_xpath('./poco:note').nil?
|
|
||||||
account.note = author_xml.at_xpath('./poco:note', poco: POCO_NS).content
|
|
||||||
end
|
|
||||||
|
|
||||||
unless author_xml.at_xpath('./xmlns:link[@rel="avatar"]').nil?
|
|
||||||
account.avatar_remote_url = author_xml.at_xpath('./xmlns:link[@rel="avatar"]').attribute('href').value
|
|
||||||
end
|
end
|
||||||
|
|
||||||
|
old_hub_url = account.hub_url
|
||||||
|
account.hub_url = hub_link['href'] if !hub_link.nil? && !hub_link['href'].blank? && (hub_link['href'] != old_hub_url)
|
||||||
account.save!
|
account.save!
|
||||||
|
|
||||||
|
SubscribeService.new.call(account) if resubscribe && (account.hub_url != old_hub_url)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -6,6 +6,7 @@ RSpec.describe Api::SalmonController, type: :controller do
|
||||||
let(:account) { Fabricate(:user, account: Fabricate(:account, username: 'catsrgr8')).account }
|
let(:account) { Fabricate(:user, account: Fabricate(:account, username: 'catsrgr8')).account }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
|
stub_request(:post, "https://pubsubhubbub.superfeedr.com/").to_return(:status => 200, :body => "", :headers => {})
|
||||||
stub_request(:get, "https://quitter.no/.well-known/host-meta").to_return(request_fixture('.host-meta.txt'))
|
stub_request(:get, "https://quitter.no/.well-known/host-meta").to_return(request_fixture('.host-meta.txt'))
|
||||||
stub_request(:get, "https://quitter.no/.well-known/webfinger?resource=acct:gargron@quitter.no").to_return(request_fixture('webfinger.txt'))
|
stub_request(:get, "https://quitter.no/.well-known/webfinger?resource=acct:gargron@quitter.no").to_return(request_fixture('webfinger.txt'))
|
||||||
stub_request(:get, "https://quitter.no/api/statuses/user_timeline/7477.atom").to_return(request_fixture('feed.txt'))
|
stub_request(:get, "https://quitter.no/api/statuses/user_timeline/7477.atom").to_return(request_fixture('feed.txt'))
|
||||||
|
|
|
@ -23,6 +23,7 @@ RSpec.describe Api::SubscriptionsController, type: :controller do
|
||||||
let(:feed) { File.read(File.join(Rails.root, 'spec', 'fixtures', 'push', 'feed.atom')) }
|
let(:feed) { File.read(File.join(Rails.root, 'spec', 'fixtures', 'push', 'feed.atom')) }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
|
stub_request(:post, "https://quitter.no/main/push/hub").to_return(:status => 200, :body => "", :headers => {})
|
||||||
stub_request(:get, "https://quitter.no/avatar/7477-300-20160211190340.png").to_return(request_fixture('avatar.txt'))
|
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/1269244").to_return(status: 404)
|
||||||
stub_request(:head, "https://quitter.no/notice/1265331").to_return(status: 404)
|
stub_request(:head, "https://quitter.no/notice/1265331").to_return(status: 404)
|
||||||
|
|
|
@ -7,6 +7,7 @@ RSpec.describe ProcessFeedService do
|
||||||
subject { ProcessFeedService.new }
|
subject { ProcessFeedService.new }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
|
stub_request(:post, "https://pubsubhubbub.superfeedr.com/").to_return(:status => 200, :body => "", :headers => {})
|
||||||
stub_request(:get, "http://kickass.zone/system/accounts/avatars/000/000/001/large/eris.png").to_return(request_fixture('avatar.txt'))
|
stub_request(:get, "http://kickass.zone/system/accounts/avatars/000/000/001/large/eris.png").to_return(request_fixture('avatar.txt'))
|
||||||
stub_request(:get, "http://kickass.zone/system/media_attachments/files/000/000/002/original/morpheus_linux.jpg?1476059910").to_return(request_fixture('attachment1.txt'))
|
stub_request(:get, "http://kickass.zone/system/media_attachments/files/000/000/002/original/morpheus_linux.jpg?1476059910").to_return(request_fixture('attachment1.txt'))
|
||||||
stub_request(:get, "http://kickass.zone/system/media_attachments/files/000/000/003/original/gizmo.jpg?1476060065").to_return(request_fixture('attachment2.txt'))
|
stub_request(:get, "http://kickass.zone/system/media_attachments/files/000/000/003/original/gizmo.jpg?1476060065").to_return(request_fixture('attachment2.txt'))
|
||||||
|
|
|
@ -1,7 +1,7 @@
|
||||||
require 'rails_helper'
|
require 'rails_helper'
|
||||||
|
|
||||||
RSpec.describe UpdateRemoteProfileService do
|
RSpec.describe UpdateRemoteProfileService do
|
||||||
let(:xml) { Nokogiri::XML(File.read(File.join(Rails.root, 'spec', 'fixtures', 'push', 'feed.atom'))).at_xpath('//xmlns:author') }
|
let(:xml) { Nokogiri::XML(File.read(File.join(Rails.root, 'spec', 'fixtures', 'push', 'feed.atom'))).at_xpath('//xmlns:feed') }
|
||||||
|
|
||||||
subject { UpdateRemoteProfileService.new }
|
subject { UpdateRemoteProfileService.new }
|
||||||
|
|
||||||
|
@ -13,7 +13,7 @@ RSpec.describe UpdateRemoteProfileService do
|
||||||
let(:remote_account) { Fabricate(:account, username: 'bob', domain: 'example.com') }
|
let(:remote_account) { Fabricate(:account, username: 'bob', domain: 'example.com') }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
subject.(xml, remote_account)
|
subject.call(xml, remote_account)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'downloads new avatar' do
|
it 'downloads new avatar' do
|
||||||
|
@ -34,10 +34,10 @@ RSpec.describe UpdateRemoteProfileService do
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'with unchanged details' do
|
context 'with unchanged details' do
|
||||||
let(:remote_account) { Fabricate(:account, username: 'bob', domain: 'example.com',display_name: 'DIGITAL CAT', note: 'Software engineer, free time musician and DIGITAL SPORTS enthusiast. Likes cats. Warning: May contain memes', avatar_remote_url: 'https://quitter.no/avatar/7477-300-20160211190340.png') }
|
let(:remote_account) { Fabricate(:account, username: 'bob', domain: 'example.com', display_name: 'DIGITAL CAT', note: 'Software engineer, free time musician and DIGITAL SPORTS enthusiast. Likes cats. Warning: May contain memes', avatar_remote_url: 'https://quitter.no/avatar/7477-300-20160211190340.png') }
|
||||||
|
|
||||||
before do
|
before do
|
||||||
subject.(xml, remote_account)
|
subject.call(xml, remote_account)
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'does not re-download avatar' do
|
it 'does not re-download avatar' do
|
||||||
|
|
Loading…
Reference in a new issue