From d8e16a226a570a7d262bdeb067273ce35b03cc7c Mon Sep 17 00:00:00 2001
From: tobi <31960611+tsmethurst@users.noreply.github.com>
Date: Sat, 17 Jun 2023 17:49:11 +0200
Subject: [chore/bugfix] Refactor `ap/extract.go` functions, return URIs more
 reliably (#1897)

---
 internal/federation/federatingprotocol.go | 20 +++++---------------
 1 file changed, 5 insertions(+), 15 deletions(-)

(limited to 'internal/federation/federatingprotocol.go')

diff --git a/internal/federation/federatingprotocol.go b/internal/federation/federatingprotocol.go
index 18feb2429..ec74de097 100644
--- a/internal/federation/federatingprotocol.go
+++ b/internal/federation/federatingprotocol.go
@@ -110,15 +110,10 @@ func (f *federator) PostInboxRequestBodyHook(ctx context.Context, r *http.Reques
 		}
 	}
 
-	// Check for TOs and CCs on the Activity.
+	// Check for TO and CC URIs on the Activity.
 	if addressable, ok := activity.(ap.Addressable); ok {
-		if toURIs, err := ap.ExtractTos(addressable); err == nil {
-			otherIRIs = append(otherIRIs, toURIs...)
-		}
-
-		if ccURIs, err := ap.ExtractCCs(addressable); err == nil {
-			otherIRIs = append(otherIRIs, ccURIs...)
-		}
+		otherIRIs = append(otherIRIs, ap.ExtractToURIs(addressable)...)
+		otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...)
 	}
 
 	// Now perform the same checks, but for the Object(s) of the Activity.
@@ -146,13 +141,8 @@ func (f *federator) PostInboxRequestBodyHook(ctx context.Context, r *http.Reques
 		}
 
 		if addressable, ok := t.(ap.Addressable); ok {
-			if toURIs, err := ap.ExtractTos(addressable); err == nil {
-				otherIRIs = append(otherIRIs, toURIs...)
-			}
-
-			if ccURIs, err := ap.ExtractCCs(addressable); err == nil {
-				otherIRIs = append(otherIRIs, ccURIs...)
-			}
+			otherIRIs = append(otherIRIs, ap.ExtractToURIs(addressable)...)
+			otherIRIs = append(otherIRIs, ap.ExtractCcURIs(addressable)...)
 		}
 	}
 
-- 
cgit v1.2.3