share: fixed another possible panic in GetShare

cernbox · Nov 4, 2024 · 189e3dd · 189e3dd
1 parent 165a578
commit 189e3dd
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/share/sql/sql.go b/share/sql/sql.go
@@ -236,8 +236,10 @@ func (m *mgr) GetShare(ctx context.Context, ref *collaboration.ShareReference) (
 		err = errtypes.NotFound(ref.String())
 	}
 
-	// resolve grantee's user type
-	s.Grantee.GetUserId().Type, _ = m.getUserType(ctx, s.Grantee.GetUserId().OpaqueId)
+	// resolve grantee's user type if applicable
+	if s.Grantee.Type == provider.GranteeType_GRANTEE_TYPE_USER {
+		s.Grantee.GetUserId().Type, _ = m.getUserType(ctx, s.Grantee.GetUserId().OpaqueId)
+	}
 
 	path, err := m.getPath(ctx, s.ResourceId)
 	if err != nil {