AP: always populate inbox and outbox in actors

required by AP! https://www.w3.org/TR/activitypub/#actor-objects

for #614

activitypub.py

@@ -648,6 +648,10 @@ def postprocess_as2_actor(actor, wrap=True):
 
     actor.update({
         'url': urls if len(urls) > 1 else urls[0],
+        # required by ActivityPub
+        # https://www.w3.org/TR/activitypub/#actor-objects
+        'inbox': g.user.ap_actor('inbox'),
+        'outbox': g.user.ap_actor('outbox'),
         # This has to be the domain for Mastodon interop/Webfinger discovery!
         # See related comment in actor() below.
         'preferredUsername': domain,

models.py

@@ -6,7 +6,6 @@
 import random
 from urllib.parse import quote, urlparse
 
-from arroba.mst import dag_cbor_cid
 from Crypto.PublicKey import ECC, RSA
 import dag_json
 from flask import g, request

tests/test_activitypub.py

@@ -1535,15 +1535,21 @@ def test_postprocess_as2_actor_attributedTo(self):
                 'id': 'baj',
                 'preferredUsername': 'site',
                 'url': 'http://localhost/r/site',
+                'inbox': 'http://bf/fake/site/ap/inbox',
+                'outbox': 'http://bf/fake/site/ap/outbox',
             },
             'attributedTo': [{
                 'id': 'bar',
                 'preferredUsername': 'site',
                 'url': 'http://localhost/r/site',
+                'inbox': 'http://bf/fake/site/ap/inbox',
+                'outbox': 'http://bf/fake/site/ap/outbox',
             }, {
                 'id': 'baz',
                 'preferredUsername': 'site',
                 'url': 'http://localhost/r/site',
+                'inbox': 'http://bf/fake/site/ap/inbox',
+                'outbox': 'http://bf/fake/site/ap/outbox',
             }],
             'to': [as2.PUBLIC_AUDIENCE],
         }, postprocess_as2({

tests/test_redirect.py

@@ -110,8 +110,6 @@ def test_as2_no_user_fetch_homepage(self, mock_get):
         del expected['endpoints']
         del expected['followers']
         del expected['following']
-        del expected['inbox']
-        del expected['outbox']
         self.assert_equals(expected, resp.json, ignore=['publicKeyPem'])
 
         self.assert_user(Web, 'user.com', direct=False, obj_as2={

tests/test_web.py

@@ -61,6 +61,8 @@
     'url': 'http://localhost/r/https://user.com/',
     'name': 'Ms. ☕ Baz',
     'preferredUsername': 'user.com',
+    'inbox': 'http://localhost/user.com/inbox',
+    'outbox': 'http://localhost/user.com/outbox',
 }
 ACTOR_AS2_USER = {
     'type': 'Person',