spacewander commented on a change in pull request #3363: URL: https://github.com/apache/apisix/pull/3363#discussion_r563560865
########## File path: apisix/plugins/fault-injection.lua ########## @@ -15,13 +15,66 @@ -- limitations under the License. -- local core = require("apisix.core") +local expr = require("resty.expr.v1") local sleep = core.sleep local random = math.random +local ipairs = ipairs local plugin_name = "fault-injection" +local vars_schema = { Review comment: I think just keep: ``` local vars_schema = { type = "array", maxItems = 20 } ``` is enough. The remain can be validated via `expr.new`. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org