Merge pull request #174 from saplingjs/feature/default-limit

Implement default limit

Author: groenroos

core/loadConfig.js

@@ -36,6 +36,7 @@ module.exports = async function (next) {
 		secret: this.utils.randString(),
 		showError: true,
 		strict: false,
+		limit: 100,
 		production: 'auto',
 		db: {
 			driver: 'Memory'

lib/Storage.js

@@ -497,13 +497,24 @@ module.exports = class Storage {
 		/* Check if we're logged in */
 		this.app.user.isUserAuthenticatedForRoute(request, response);
 
+		/* Parse max limit */
+		const limit = (this.config.limit && this.config.limit > 0) ? this.config.limit : false;
+
 		/* Parse limit options */
 		if ('limit' in request.query) {
 			options.limit = Number(request.query.limit) || null;
 
+			/* If max limit is set in config, ensure we don't go over it */
+			if (limit && options.limit > limit) {
+				options.limit = limit;
+			}
+
 			if ('skip' in request.query) {
 				options.skip = Number(request.query.skip) || null;
 			}
+		} else if (limit) {
+			/* If max limit is set in config, enforce it */
+			options.limit = limit;
 		}
 
 		/* Parse sorting option */