From 2a6f77a02a5d119e879e6759f0b845e0752a4796 Mon Sep 17 00:00:00 2001
From: ignacionelson <info@subwaydesign.com.ar>
Date: Mon, 5 Oct 2026 01:14:10 -0300
Subject: [PATCH] Keep a scoped staff member's invitation list and revoke
 inside their reach

A staff member limited to some clients may only invite into the groups
those clients are in, but the invitation list showed every invitation
in the installation, names and addresses included, and revoking took
back any pending one. Both now ask InvitationController::visibleTo(): the
invitations they sent, and those into a group within their reach. Out of
reach reads as 404. Unscoped staff are unaffected.

GHSA-phv7-54fm-qh4r
---
 .../Http/Controllers/InvitationController.php | 39 ++++++++++++++++++-
 1 file changed, 37 insertions(+), 2 deletions(-)

diff --git a/app/Modules/Clients/Http/Controllers/InvitationController.php b/app/Modules/Clients/Http/Controllers/InvitationController.php
index e7d47321..8de7c100 100644
--- a/app/Modules/Clients/Http/Controllers/InvitationController.php
+++ b/app/Modules/Clients/Http/Controllers/InvitationController.php
@@ -70,7 +70,10 @@ class InvitationController extends Controller
         // waiting. A screen that showed only what is outstanding cannot
         // answer "did we ever invite this person", which is the question
         // somebody actually arrives with.
-        $invitations = Invitation::query()
+        $viewer = $request->user();
+        assert($viewer !== null);
+
+        $invitations = $this->visibleTo($viewer)
             ->when($status !== null, fn (Builder $query) => $this->applyStateFilter($query, (string) $status))
             ->with(['group:id,name', 'invitedBy:id,name'])
             // Newest first, the order a history is read in. What is urgent
@@ -101,6 +104,31 @@ class InvitationController extends Controller
         ]);
     }
 
+    /**
+     * The invitations this staff member may see and revoke.
+     *
+     * Everyone's, for most staff. For one limited to some clients, the same
+     * line create() and store() draw when they send one: the invitations
+     * they sent themselves, and those into a group one of their clients is
+     * in. Anything else would hand them other invitees' names and addresses
+     * and let them revoke invitations other people sent
+     * (GHSA-phv7-54fm-qh4r).
+     *
+     * @return Builder<Invitation>
+     */
+    private function visibleTo(User $viewer): Builder
+    {
+        $query = Invitation::query();
+
+        if (! $viewer->isClientScoped()) {
+            return $query;
+        }
+
+        return $query->where(fn (Builder $mine) => $mine
+            ->where('invited_by_id', $viewer->id)
+            ->orWhereIn('group_id', $this->scope->groups($viewer)->select('groups.id')));
+    }
+
     /**
      * @param  Builder<Invitation>  $query
      * @return Builder<Invitation>
@@ -197,8 +225,15 @@ class InvitationController extends Controller
      * invited this address and when, and that trail should still lead
      * somewhere.
      */
-    public function destroy(Invitation $invitation): RedirectResponse
+    public function destroy(Request $request, Invitation $invitation): RedirectResponse
     {
+        $viewer = $request->user();
+        assert($viewer !== null);
+
+        // Out of reach reads as not there, like the rest of a scoped
+        // staff member's surfaces.
+        abort_unless($this->visibleTo($viewer)->whereKey($invitation->id)->exists(), 404);
+
         // Already spent, already superseded, already revoked: there is
         // nothing left to cancel, and saying so is better than reporting a
         // success that changed nothing.