Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plafonnement du limit ? #198

Closed
lpofredc opened this issue Aug 29, 2023 · 3 comments · Fixed by #212
Closed

Plafonnement du limit ? #198

lpofredc opened this issue Aug 29, 2023 · 3 comments · Fixed by #212

Comments

@lpofredc
Copy link
Contributor

Bonjour, y a-t-il une raison particulière de plafonner le limit à 1000 résultats ?

if limit > 1000:
limit = 1000

@camillemonchicourt
Copy link
Member

Peut-être des questions initiales de performance ou pour garantir que l'API va pas être trop chargée par un usage qui ne serait pas souhaitable et mettrait a plat le serveur.

Mais on pourrait peut-être plutôt mettre une limite à 1000 par défaut quand elle n'est pas spécifiée mais permettre d'aller à plus.
Peut-être quand même quand même garder un max à 10.000 ou plus ? Et le rendre paramétrable pour ceux qui voudraient permettre un usage avec une limite plus grande ?

@TheoLechemia
Copy link
Member

Mais on pourrait peut-être plutôt mettre une limite à 1000 par défaut quand elle n'est pas spécifiée mais permettre d'aller à plus.

Oui c'est un fonctionnement assez courant et logique

@camillemonchicourt
Copy link
Member

Ajout des paramètres dans la 1.7.1 :

  • max_page_size_api : Max number of items for each API call (default is 1000)
  • default_page_size_api : Default number of items returned on each API call (default is 1000)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants