nic-6443 commented on code in PR #12551:
URL: https://github.com/apache/apisix/pull/12551#discussion_r2351677725


##########
apisix/cli/schema.lua:
##########
@@ -259,6 +259,14 @@ local config_schema = {
                     default = false,
                     description = "a global switch to disable upstream health 
checks",
                 },
+                trusted_addresses = {

Review Comment:
   The reason for this error is that there is no openresty environment in the 
apisix cli, which makes it impossible to use the `resty.lru` library. 
   I have considered the dependency relationship between `schema_def.lua` and 
`schema.lua`, and I think their dependency relationship is reversed. It should 
be that `schema.lua` depends on `schema_def.lua`, allowing `schema_def.lua` to 
retain only the definition of schema, in line with the naming of this file. 
   Added a new commit to this PR to fix this dependency issue: 
https://github.com/apache/apisix/pull/12551/commits/30ce9c26e65f9e8ecfc063c08abe9b94b75bd9ba



-- 
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.

To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to