Set maximum LIMIT value.

raintank · Apr 3, 2017 · 1dd2330 · 1dd2330
1 parent 0ed1d0d
commit 1dd2330
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/dist/query_builder.js b/dist/query_builder.js
diff --git a/dist/query_builder.js.map b/dist/query_builder.js.map
diff --git a/dist/query_builder.ts b/dist/query_builder.ts
@@ -2,6 +2,9 @@
 
 import _ from 'lodash';
 
+// Maximum LIMIT value
+let MAX_LIMIT = 500000000;
+
 export class CrateQueryBuilder {
   schema: string;
   table: string;
@@ -146,6 +149,7 @@ export class CrateQueryBuilder {
     query += " ASC";
 
     if (limit) {
+      limit = Math.min(limit, MAX_LIMIT);
       query += ` LIMIT ${limit}`;
     }
 
@@ -199,6 +203,7 @@ export class CrateQueryBuilder {
     query += " ASC";
 
     if (limit) {
+      limit = Math.min(limit, MAX_LIMIT);
       query += ` LIMIT ${limit}`;
     }
 

diff --git a/src/query_builder.ts b/src/query_builder.ts
@@ -2,6 +2,9 @@
 
 import _ from 'lodash';
 
+// Maximum LIMIT value
+let MAX_LIMIT = 500000000;
+
 export class CrateQueryBuilder {
   schema: string;
   table: string;
@@ -146,6 +149,7 @@ export class CrateQueryBuilder {
     query += " ASC";
 
     if (limit) {
+      limit = Math.min(limit, MAX_LIMIT);
       query += ` LIMIT ${limit}`;
     }
 
@@ -199,6 +203,7 @@ export class CrateQueryBuilder {
     query += " ASC";
 
     if (limit) {
+      limit = Math.min(limit, MAX_LIMIT);
       query += ` LIMIT ${limit}`;
     }