Project

General

Profile

Développement #103683

API : avoir une options pour ne pas ramener les rôles

Added by Guillaume Baffoin 30 days ago. Updated 19 days ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
20 March 2025
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

precision : sur le endpoint /api/users/

Quelques fois, on cherche juste un utilisateur et pas forcément son rôle.
Lorsque l'on fait ce genre de recherche, pouvoir ne pas récupérer les rôles permet de ne pas surcharger de requêtes inutiles la base de données et de retourner seulement la liste des utilisateurs.

(on pourrait même dire que ça devrait être le comportement par défaut, mais c'est un autre sujet)


Related issues

Related to Authentic 2 - Développement #103784: API: optimiser les listing des utilisateursSolution déployée23 March 2025

Actions

Associated revisions

Revision 783053b9 (diff)
Added by Benjamin Dauvergne 22 days ago

api: add an include-roles flag to user listing APIs (#103683)

History

#3

Updated by Thomas Noël 30 days ago

Précision : il s'agit de /api/users/ qui, pour chaque utilisateur, cherche ses rôles. Quand un limit=100 est ajouté, cela remonte les rôles de 100 utilisateurs et c'est très consommateur.

On aimerait pouvoir faire un /api/users/?q=test&roles=false&limit=100

Et sans doute que ça pourrait être le comportement par défaut sans que ça impacte rien (mais c'est peut-être un autre sujet).

#4

Updated by Guillaume Baffoin 30 days ago

  • Description updated (diff)
#5

Updated by Benjamin Dauvergne 29 days ago

Ça ne devrait pas coûter si cher de récupérer les rôles de 100 utilisateurs:
1. vérifier que la façon dont c'est fait est bien la plus efficace (1/2/3 select pour tous les utilisateurs, pas une série de select par utilisateur)
2. éventuellement avoir roles=false par défaut effectivement (mais comme ça a plus d'effets de bord potentiels ça me semble plus sûr de faire 1. d'abord)

#6

Updated by Benjamin Dauvergne 29 days ago

Vérifier aussi que la pagination est obligatoire et limitée (genre limit = max(limit, 200)).

#9

Updated by Benjamin Dauvergne 27 days ago

#11

Updated by Benjamin Dauvergne 25 days ago

  • Assignee set to Benjamin Dauvergne
#12

Updated by Robot Gitea 25 days ago

  • Status changed from Nouveau to En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#13

Updated by Robot Gitea 25 days ago

  • Status changed from En cours to Solution proposée
#14

Updated by Robot Gitea 23 days ago

  • Status changed from Solution proposée to Solution validée

Paul Marillonnet (pmarillonnet) a approuvé une pull request sur Gitea concernant cette demande :

#15

Updated by Robot Gitea 22 days ago

  • Status changed from Solution validée to En cours

Benjamin Dauvergne (bdauvergne) a fermé une pull request sur Gitea concernant cette demande.

#16

Updated by Robot Gitea 22 days ago

  • Status changed from En cours to Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#17

Updated by Robot Gitea 21 days ago

  • Status changed from Solution proposée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#18

Updated by Transition automatique 19 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF