Repository navigation
perf(users): identifiants des autres stratégies chargés au dépliage (G-132) - #1250
Merged
Merged
Conversation
…G-132) Retour de QA : la liste des utilisateurs mettait plusieurs secondes à s'ouvrir sur un backend distant. Elle appelait getCredentials pour chaque utilisateur et chaque stratégie, en série. Elle ne charge plus que `local`, seul affiché ligne repliée, en parallèle ; UserItem charge toutes les stratégies à son premier dépliage. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Retour de QA : Security → Users met plusieurs secondes à s'ouvrir sur paas.
Cause :
performSearchUsersappelaitgetCredentialspour chaque utilisateur × stratégie, en série (2 + 25 × 2 allers-retours pour 25 utilisateurs et deux stratégies). Invisible en local (G-132).Correctif
local(seul affiché ligne repliée), en parallèle, etgetStrategiesen parallèle de la recherche.UserItemcharge toutes les stratégies à son premier dépliage (getUserCredentials), et oublie ce chargement si la liste recharge ses documents.users: le dépliage rend les identifiants.Vérifié contre un build sur stack neuve :
users21/21,apiKeys7/7,profiles7/7.4-devdans une PR séparée.🤖 Generated with Claude Code