Ankit-cs commented on PR #14040: URL: https://github.com/apache/apisix/pull/14040#issuecomment-6309176642
Hi @ruanweihong-aaa, I have updated the PR to handle `$upstream_uri` during `data-mask`'s log phase: 1. `data-mask.lua`: When `query_masked` is active in `_M.log`, we now inspect `ctx.var.upstream_uri`. If present with query parameters, we re-serialize `upstream_uri` using the sanitized arguments (or strip the query delimiter `?` completely if all query arguments were masked). 2. Regression Test: Added tests 8–11 in `t/router/servlet-upstream-uri.t` verifying that with `normalize_uri_like_servlet: true`, `data-mask` sanitizes both `$request_line` and `$upstream_uri` in the log output for both single and multiple query parameters. Could you please take another look? -- 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]
