AlinsRan commented on PR #12885:
URL: https://github.com/apache/apisix/pull/12885#issuecomment-4910323909
Added the test (pushed to the PR branch). Followed @nic-6443's suggestion —
since #13078 substitutes env vars in the raw `apisix.yaml` text before parsing,
the original issue repro no longer reaches `resolve_conf_var`, so the test
targets a `config.yaml` key instead: an env var used as a
`custom_lua_shared_dict` name. It asserts the resolved dict is rendered and the
stale `${{DICT_NAME}}` placeholder is gone from `nginx.conf` — fails without
this fix, passes with it. Thanks @nic-6443 for the pointer.
--
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]
