Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
rainie_yuyue
soar
提交
9443120c
S
soar
项目概览
rainie_yuyue
/
soar
与 Fork 源项目一致
Fork自
Xiaomi / soar
通知
1
Star
1
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
S
soar
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
9443120c
编写于
6月 28, 2021
作者:
martianzhang
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
fix #273
add test cases comments use english more understandable
上级
6ae5c3f5
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
33 addition
and
12 deletion
+33
-12
advisor/heuristic.go
advisor/heuristic.go
+9
-8
advisor/heuristic_test.go
advisor/heuristic_test.go
+24
-4
未找到文件。
advisor/heuristic.go
浏览文件 @
9443120c
...
...
@@ -133,16 +133,17 @@ func (q *Query4Audit) RuleEqualLike() Rule {
if
strings
.
ToLower
(
expr
.
Operator
)
==
"like"
{
switch
sqlval
:=
expr
.
Right
.
(
type
)
{
case
*
sqlparser
.
SQLVal
:
// not start with '%', '_' && not end with '%', '_'
// 1. string that not contain '%', '_'
// 2. int, bit, float without wildcard
var
hasWildCard
bool
if
sqlval
.
Type
==
0
{
if
sqlval
.
Val
[
0
]
!=
0x25
&&
sqlval
.
Val
[
0
]
!=
0x5f
&&
sqlval
.
Val
[
len
(
sqlval
.
Val
)
-
1
]
!=
0x5f
&&
sqlval
.
Val
[
len
(
sqlval
.
Val
)
-
1
]
!=
0x25
{
rule
=
HeuristicRules
[
"ARG.002"
]
return
false
,
nil
for
_
,
sqlElem
:=
range
sqlval
.
Val
{
if
sqlElem
==
0x25
||
sqlElem
==
0x5f
{
hasWildCard
=
true
}
}
}
else
{
}
if
!
hasWildCard
{
rule
=
HeuristicRules
[
"ARG.002"
]
return
false
,
nil
}
...
...
advisor/heuristic_test.go
浏览文件 @
9443120c
...
...
@@ -132,11 +132,18 @@ func TestRulePrefixLike(t *testing.T) {
// ARG.002
func
TestRuleEqualLike
(
t
*
testing
.
T
)
{
common
.
Log
.
Debug
(
"Entering function: %s"
,
common
.
GetFunctionName
())
sqls
:=
[]
string
{
"select col from tbl where id like 'abc'"
,
"select col from tbl where id like 1"
,
sqls
:=
[][]
string
{
{
"select col from tbl where id like 'abc'"
,
"select col from tbl where id like 1"
,
},
{
"select col from tbl where id like 'abc%'"
,
"select col from tbl where id like '%abc'"
,
"select col from tbl where id like 'a%c'"
,
// issue #273
},
}
for
_
,
sql
:=
range
sqls
{
for
_
,
sql
:=
range
sqls
[
0
]
{
q
,
err
:=
NewQuery4Audit
(
sql
)
if
err
==
nil
{
rule
:=
q
.
RuleEqualLike
()
...
...
@@ -147,6 +154,19 @@ func TestRuleEqualLike(t *testing.T) {
t
.
Error
(
"sqlparser.Parse Error:"
,
err
)
}
}
for
_
,
sql
:=
range
sqls
[
1
]
{
q
,
err
:=
NewQuery4Audit
(
sql
)
if
err
==
nil
{
rule
:=
q
.
RuleEqualLike
()
if
rule
.
Item
==
"ARG.002"
{
t
.
Error
(
"Rule not match:"
,
rule
.
Item
,
"Expect : OK"
)
}
}
else
{
t
.
Error
(
"sqlparser.Parse Error:"
,
err
)
}
}
common
.
Log
.
Debug
(
"Exiting function: %s"
,
common
.
GetFunctionName
())
}
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录