From 4cb1d42ba2ffbd31c5e865e55c408363ca35f47a Mon Sep 17 00:00:00 2001
From: multiple creatures <dev@multiple-creature.party>
Date: Fri, 21 Feb 2020 07:30:29 -0600
Subject: fix invisible mode being unfederatable, mask actor props & outbox
 instead of actor itself

---
 .../concerns/account_controller_concern.rb           |  2 +-
 app/serializers/activitypub/actor_serializer.rb      | 20 ++++++++++----------
 2 files changed, 11 insertions(+), 11 deletions(-)

(limited to 'app')

diff --git a/app/controllers/concerns/account_controller_concern.rb b/app/controllers/concerns/account_controller_concern.rb
index 4fd9af688..a195c27de 100644
--- a/app/controllers/concerns/account_controller_concern.rb
+++ b/app/controllers/concerns/account_controller_concern.rb
@@ -10,7 +10,7 @@ module AccountControllerConcern
   included do
     layout 'public'
 
-    before_action :check_account_hidden
+    before_action :check_account_hidden, if: -> { request.format.nil? || request.format == :html }
     before_action :set_instance_presenter
     before_action :set_link_headers, if: -> { request.format.nil? || request.format == :html }
   end
diff --git a/app/serializers/activitypub/actor_serializer.rb b/app/serializers/activitypub/actor_serializer.rb
index 81a41b917..2a61d8026 100644
--- a/app/serializers/activitypub/actor_serializer.rb
+++ b/app/serializers/activitypub/actor_serializer.rb
@@ -94,19 +94,19 @@ class ActivityPub::ActorSerializer < ActivityPub::Serializer
   end
 
   def name
-    object.display_name
+    object.hidden? ? '' : object.display_name
   end
 
   def summary
-    Formatter.instance.simplified_format(object)
+    object.hidden? ? '' : Formatter.instance.simplified_format(object)
   end
 
   def icon
-    object.avatar
+    object.hidden? ? nil : object.avatar
   end
 
   def image
-    object.header
+    object.hidden? ? nil : object.header
   end
 
   def public_key
@@ -118,11 +118,11 @@ class ActivityPub::ActorSerializer < ActivityPub::Serializer
   end
 
   def avatar_exists?
-    object.avatar?
+    object.hidden? ? false : object.avatar?
   end
 
   def header_exists?
-    object.header?
+    object.hidden? ? false : object.header?
   end
 
   def manually_approves_followers
@@ -134,15 +134,15 @@ class ActivityPub::ActorSerializer < ActivityPub::Serializer
   end
 
   def virtual_tags
-    object.emojis + object.tags
+    object.hidden? ? nil : (object.emojis + object.tags)
   end
 
   def virtual_attachments
-    object.fields
+    object.hidden? ? nil : object.fields
   end
 
   def moved_to
-    ActivityPub::TagManager.instance.uri_for(object.moved_to_account)
+    object.hidden? ? nil : ActivityPub::TagManager.instance.uri_for(object.moved_to_account)
   end
 
   def also_known_as?
@@ -182,7 +182,7 @@ class ActivityPub::ActorSerializer < ActivityPub::Serializer
     end
 
     def value
-      Formatter.instance.format_field(object.account, object.value)
+      object.account.hidden? ? '' : Formatter.instance.format_field(object.account, object.value)
     end
   end
 end
-- 
cgit