Développement #103683
API : avoir une options pour ne pas ramener les rôles
0%
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
Associated revisions
History
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).
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)
Updated by Benjamin Dauvergne 29 days ago
Vérifier aussi que la pagination est obligatoire et limitée (genre limit = max(limit, 200)).
Updated by Benjamin Dauvergne 27 days ago
- Related to Développement #103784: API: optimiser les listing des utilisateurs added
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/483
- Titre : WIP: api: add an include-roles flag to user listing APIs (#103683)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/483/files
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 :
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.
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/484
- Titre : api: add an include-roles flag to user listing APIs (#103683)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/484/files
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/484
- Titre : api: add an include-roles flag to user listing APIs (#103683)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/484/files
Updated by Transition automatique 19 days ago
- Status changed from Résolu (à déployer) to Solution déployée
api: add an include-roles flag to user listing APIs (#103683)