Clarify source comment.

wireapp · Dec 20, 2024 · 587052a · 587052a
1 parent bc88b86
commit 587052a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/services/galley/src/Galley/API/LegalHold/Conflicts.hs b/services/galley/src/Galley/API/LegalHold/Conflicts.hs
@@ -96,8 +96,9 @@ guardLegalholdPolicyConflicts (ProtectedUser self) otherClients = do
   case view (settings . featureFlags . to npProject) opts of
     FeatureLegalHoldDisabledPermanently -> case FutureWork @'LegalholdPlusFederationNotImplemented () of
       FutureWork () ->
-        -- FUTUREWORK: if federation is enabled, we still need to run the guard!
-        -- see also: LegalholdPlusFederationNotImplemented
+        -- FUTUREWORK: once we support federation and LH in combination, we still need to run
+        -- the guard to protect against other federating instances running LH!  see also:
+        -- LegalholdPlusFederationNotImplemented
         pure ()
     FeatureLegalHoldDisabledByDefault -> guardLegalholdPolicyConflictsUid self otherClients
     FeatureLegalHoldWhitelistTeamsAndImplicitConsent -> guardLegalholdPolicyConflictsUid self otherClients