AlinsRan commented on PR #13713: URL: https://github.com/apache/apisix/pull/13713#issuecomment-5019005045
Verified the two claims this change rests on directly against nginx, outside the test suite (OpenResty 1.29.2.4): A minimal conf with the quoted form passes `nginx -t`: ```nginx env "TEST_SPACE=a b"; env "TEST_QUOTE=a\"b"; env "TEST_BACKSLASH=a\\b"; ``` Switching just the first line back to the unquoted form that master renders today reproduces the issue exactly: ``` nginx: [emerg] invalid number of arguments in "env" directive in .../nginx.conf:5 ``` And the values arrive intact in the worker — `os.getenv` returns: ``` SPACE=[a b] QUOTE=[a"b] BS=[a\b] ``` So the escaping round-trips: nginx unescapes the quoted token back to the original value, confirming there is no `$` handling needed and that the reported "nginx limitation" is not one. The `t/cli` suite itself has not been run locally; that part is on CI. -- 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]
