lxbme commented on code in PR #13078:
URL: https://github.com/apache/apisix/pull/13078#discussion_r2903436495
##########
apisix/core/config_yaml.lua:
##########
@@ -153,7 +153,7 @@ local function update_config(table, conf_version)
return
end
- local ok, err = file.resolve_conf_var(table)
+ local ok, err = file.resolve_conf_var(table, false)
Review Comment:
According to @Baoyuantop's analysis in the original issue, config.yaml and
apisix.yaml serve different use cases. config.yaml is used for APISIX core
configurations (which require type conversion), whereas apisix.yaml is used for
route/plugin configurations in standalone mode (where strings are typically
expected). Completely disabling type conversion for apisix.yaml is the expected
behavior.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
