Skip to content

Commit

Permalink
Update autoindex level check (#687)
Browse files Browse the repository at this point in the history
only make sure level > 0, do not check range, range check should be done
in the server side

Signed-off-by: chasingegg <chao.gao@zilliz.com>
  • Loading branch information
chasingegg authored Mar 29, 2024
1 parent 7ab04e8 commit cb6426e
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 13 deletions.
3 changes: 1 addition & 2 deletions entity/genidx/genidx.go
Original file line number Diff line number Diff line change
Expand Up @@ -622,7 +622,7 @@ func main() {
SearchParams: []idxParam{
{
Name: "level",
ValidationRule: "[1, 3]",
ValidationRule: "[1, 9223372036854775807]",
},
},
ValidExamples: []string{
Expand All @@ -634,7 +634,6 @@ func main() {
},
InvalidSearchParams: []string{
"level = 0",
"level = 10",
"level = -1",
},
},
Expand Down
6 changes: 3 additions & 3 deletions entity/indexes_search_param_gen.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 1 addition & 8 deletions entity/indexes_search_param_gen_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit cb6426e

Please sign in to comment.