modify sql size

Xibao-Lv · Aug 28, 2013 · b6e4c31 · b6e4c31
1 parent d36a6ea
commit b6e4c31
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/main/java/com/alibaba/druid/wall/WallProvider.java b/src/main/java/com/alibaba/druid/wall/WallProvider.java
@@ -58,15 +58,15 @@ public abstract class WallProvider {
     private boolean                                       whiteListEnable         = true;
     private LRUCache<String, WallSqlStat>                 whiteList;
 
-    private int                                           MAX_SQL_LENGTH          = 2048;                                              // 1k
+    private int                                           MAX_SQL_LENGTH          = 8192;                                              // 8k
 
-    private int                                           whiteSqlMaxSize         = 1000;                                               // 1k
+    private int                                           whiteSqlMaxSize         = 1000;
 
     private boolean                                       blackListEnable         = true;
     private LRUCache<String, WallSqlStat>                 blackList;
     private LRUCache<String, WallSqlStat>                 blackMergedList;
 
-    private int                                           blackSqlMaxSize         = 200;                                               // 1k
+    private int                                           blackSqlMaxSize         = 200;
 
     protected final WallConfig                            config;
 
@@ -616,6 +616,7 @@ private WallCheckResult checkInternal(String sql) {
         final List<Violation> violations = new ArrayList<Violation>();
         List<SQLStatement> statementList = new ArrayList<SQLStatement>();
         boolean syntaxError = false;
+        boolean illegal = false;
         try {
             SQLStatementParser parser = createParser(sql);
             parser.getLexer().setCommentHandler(WallCommentHandler.instance);
@@ -630,17 +631,19 @@ private WallCheckResult checkInternal(String sql) {
             if (lastToken != Token.EOF) {
                 violations.add(new IllegalSQLObjectViolation(ErrorCode.SYNTAX_ERROR, "not terminal sql, token "
                                                                                      + lastToken, sql));
+                illegal = true;
             }
         } catch (NotAllowCommentException e) {
             violations.add(new SyntaxErrorViolation(e, sql));
             incrementCommentDeniedCount();
         } catch (ParserException e) {
             syntaxErrrorCount.incrementAndGet();
-            syntaxError = true;
+            illegal = syntaxError = true;
             if (config.isStrictSyntaxCheck()) {
                 violations.add(new SyntaxErrorViolation(e, sql));
             }
         } catch (Exception e) {
+            illegal = true;
             violations.add(new SyntaxErrorViolation(e, sql));
         }
 
@@ -655,6 +658,7 @@ private WallCheckResult checkInternal(String sql) {
             try {
                 stmt.accept(visitor);
             } catch (ParserException e) {
+                illegal = true;
                 violations.add(new SyntaxErrorViolation(e, sql));
             }
         }