Merge pull request #110 from kmycode/kbtopic-82-friend-server-inbox-url

Change: #82 フレンド申請時、自分のサーバーのInbox URLを伝える

app/lib/activitypub/activity/follow.rb

@@ -53,7 +53,7 @@ def request_follow_for_friend
       friend.update!(passive_state: :pending, active_state: :idle, passive_follow_activity_id: @json['id'])
     else
       @friend = FriendDomain.new(domain: @account.domain, passive_state: :pending, passive_follow_activity_id: @json['id'])
-      @friend.initialize_inbox_url!
+      @friend.inbox_url = @json['inboxUrl'].presence || @friend.default_inbox_url
       @friend.save!
     end
 

app/models/friend_domain.rb

@@ -93,16 +93,12 @@ def destroy_without_signal!
     destroy!
   end
 
-  def initialize_inbox_url!
-    self.inbox_url = default_inbox_url
-  end
-
-  private
-
   def default_inbox_url
     "https://#{domain}/inbox"
   end
 
+  private
+
   def delete_for_friend!
     activity_id = ActivityPub::TagManager.instance.generate_uri_for(nil)
     payload     = Oj.dump(delete_follow_activity(activity_id))
@@ -118,6 +114,9 @@ def follow_activity(activity_id)
       type: 'Follow',
       actor: ActivityPub::TagManager.instance.uri_for(some_local_account),
       object: ActivityPub::TagManager::COLLECTIONS[:public],
+
+      # Cannot use inbox_url method because this model also has inbox_url column
+      inboxUrl: "https://#{Rails.configuration.x.web_domain}/inbox",
     }
   end
 

spec/lib/activitypub/activity/follow_spec.rb

@@ -310,6 +310,7 @@
     let!(:friend) { Fabricate(:friend_domain, domain: 'abc.com', passive_state: :idle) }
     let!(:owner_user) { Fabricate(:user, role: UserRole.find_by(name: 'Owner')) }
     let!(:patch_user) { Fabricate(:user, role: Fabricate(:user_role, name: 'OhagiOps', permissions: UserRole::FLAGS[:manage_federation])) }
+    let(:inbox_url) { nil }
 
     let(:json) do
       {
@@ -318,6 +319,7 @@
         type: 'Follow',
         actor: ActivityPub::TagManager.instance.uri_for(sender),
         object: 'https://www.w3.org/ns/activitystreams#Public',
+        inboxUrl: inbox_url,
       }.with_indifferent_access
     end
 
@@ -343,6 +345,24 @@
       end
     end
 
+    context 'when no record and inbox_url is specified' do
+      let(:inbox_url) { 'https://ohagi.com/inbox' }
+
+      before do
+        friend.update(domain: 'def.com')
+      end
+
+      it 'marks the friend as pending' do
+        subject.perform
+
+        friend = FriendDomain.find_by(domain: 'abc.com')
+        expect(friend).to_not be_nil
+        expect(friend.they_are_pending?).to be true
+        expect(friend.passive_follow_activity_id).to eq 'foo'
+        expect(friend.inbox_url).to eq 'https://ohagi.com/inbox'
+      end
+    end
+
     context 'when my server is pending' do
       before do
         friend.update(active_state: :pending)

spec/models/friend_domain_spec.rb

@@ -21,6 +21,7 @@
         type: 'Follow',
         actor: 'https://cb6e6126.ngrok.io/actor',
         object: 'https://www.w3.org/ns/activitystreams#Public',
+        inboxUrl: 'https://cb6e6126.ngrok.io/inbox',
       }))).to have_been_made.once
     end
   end