Take limit in ListOptions

Is your feature request related to a problem? Please describe. When running a Vendure instance that contains many Products and ProductVariants, it is currently possible to query all products at once as there is no limit for the take input. This is obviously not desired as this can easily bring down the server.

Describe the solution you’d like Allow to define a limit and default value for the take variable in ListOptions and SearchInput.

Describe alternatives you’ve considered

• Define a Custom Scalar with a maximum and default value.
• Other options?

[michaelbromley]

Got it, thanks for the explanation. One option to make the limit more explicit would be to throw a UserInputError when attempting to take too many.

[alex-dwt]

Thanks for a full description @michaelbromley Initially I’ve read the text in the 1st message:

Describe the solution you’d like Allow to define a limit and default value for the take variable in ListOptions and SearchInput.

And because of SearchInput I decided that you are going to improve search_query also, and start thinking that you probably missed this -) PS. But maybe it makes some sense to “protect” search_query also, or maybe not, anyway I agree it’s not so important now