ActivityPub.postprocess_as2: always populate images into attachments …

…as objects

hopefully for #1000 (comment), ATProto images without alt text not showing up in AP

activitypub.py

@@ -685,7 +685,8 @@ def postprocess_as2(activity, orig_obj=None, wrap=True):
     imgs = util.get_list(obj_or_activity, 'image')
     if imgs:
         atts = obj_or_activity['attachment']
-        atts.extend(img for img in imgs if img not in atts)
+        atts.extend((img if isinstance(img, dict) else {'url': img})
+                    for img in imgs if img not in atts)
 
     # cc target's author(s), recipients, mentions
     # https://www.w3.org/TR/activitystreams-vocabulary/#audienceTargeting

tests/test_activitypub.py

@@ -1950,6 +1950,17 @@ def test_postprocess_as2_multiple_image(self):
             'image': [{'url': 'http://r/foo'}, {'url': 'http://r/bar'}],
         }))
 
+    def test_postprocess_as2_image_bare_string_url(self):
+        self.assert_equals({
+            'id': 'http://localhost/r/xyz',
+            'attachment': [{'url': 'http://r/foo'}],
+            'image': ['http://r/foo'],
+            'to': [as2.PUBLIC_AUDIENCE],
+        }, postprocess_as2({
+            'id': 'xyz',
+            'image': ['http://r/foo'],
+        }))
+
     def test_postprocess_as2_note(self):
         self.assert_equals({
             'id': 'http://localhost/r/xyz',