Resolve remote accounts when mentioned even if they are already known (#5539)

This commit reduces the risk of not having up-to-date public key or protocol
information for a remote account, which is required to deliver toots
(especially direct messages).
This commit is contained in:
ThibG 2017-11-07 14:31:57 +01:00 committed by Eugen Rochko
parent 47b0c61853
commit 7bea1530f4

View File

@ -11,16 +11,11 @@ class ProcessMentionsService < BaseService
return unless status.local? return unless status.local?
status.text.scan(Account::MENTION_RE).each do |match| status.text.scan(Account::MENTION_RE).each do |match|
username, domain = match.first.split('@')
mentioned_account = Account.find_remote(username, domain)
if mentioned_account.nil? && !domain.nil?
begin begin
mentioned_account = resolve_remote_account_service.call(match.first.to_s) mentioned_account = resolve_remote_account_service.call(match.first.to_s)
rescue Goldfinger::Error, HTTP::Error rescue Goldfinger::Error, HTTP::Error
mentioned_account = nil mentioned_account = nil
end end
end
next if mentioned_account.nil? next if mentioned_account.nil?