fix: improve instropection perfomance

prefetch client resource server

Author: smarcet

app/Repositories/DoctrineOAuth2ClientRepository.php

@@ -12,6 +12,7 @@
  * limitations under the License.
  **/
 
+use Doctrine\ORM\Query;
 use Doctrine\ORM\QueryBuilder;
 use Models\OAuth2\Client;
 use OAuth2\Repositories\IClientRepository;
@@ -103,18 +104,29 @@ public function getClientById(string $client_id):?Client
      * @return Client|null
      * @throws \Doctrine\ORM\NonUniqueResultException
      */
-    public function getClientByIdCacheable(string $client_id):?Client
+    public function getClientByIdCacheable(string $client_id, bool $withResourceServer = true):?Client
     {
         return $this->getEntityManager()
             ->createQueryBuilder()
             ->select("c")
             ->from($this->getBaseEntity(), "c")
             ->where("c.client_id = (:client_id)")
             ->setParameter("client_id", trim($client_id))
-            ->setMaxResults(1)
-            ->getQuery()
-            ->setCacheable(true)
-            ->getOneOrNullResult();
+            ->setMaxResults(1);
+
+        if ($withResourceServer) {
+            // fetch join dirigido
+            $qb->addSelect('rs')
+                ->leftJoin('c.resource_server', 'rs'); // JOIN FETCH implícito por el addSelect + asociación
+        }
+
+        $q = $qb->getQuery();
+
+        $q->useQueryCache(true);
+        $q->enableResultCache(600, 'client_by_id_'.$client_id); // TTL 10 min
+        $q->setHint(Query::HINT_READ_ONLY, true);
+
+        return $q->getOneOrNullResult();
     }
 
     /**

app/libs/OAuth2/Repositories/IClientRepository.php

@@ -33,10 +33,10 @@ public function getClientById(string $client_id):?Client;
 
     /**
      * @param string $client_id
+     * @param bool $withResourceServer
      * @return Client|null
-     * @throws \Doctrine\ORM\NonUniqueResultException
      */
-    public function getClientByIdCacheable(string $client_id):?Client;
+    public function getClientByIdCacheable(string $client_id, bool $withResourceServer = true):?Client;
 
     /**
      * @param int $id