mrhhsg opened a new pull request, #69506:
URL: https://github.com/apache/doris/pull/69506
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary: `HttpRequest::debug_string()` rendered the full request URI
and every parsed parameter key and value, so query credentials could appear in
BE diagnostics. The query-parse failure warning also included the raw query.
Render the parsed path and parameter count instead, and omit query text from
that warning. Request parsing and URI/parameter accessors retain their existing
behavior. This change is limited to these logging paths; action-specific HTTP
logs are unchanged.
### Release note
BE `HttpRequest::debug_string()` no longer emits query strings or parameter
keys and values. Query-parse failure warnings no longer emit the raw query.
### Check List (For Author)
- Test:
- Unit Test: `HttpRequestTest.*` passed (9/9). The unmodified full BE UT
build was blocked by a pre-existing compile error in an unrelated test
translation unit; the focused suite was run with that unit temporarily excluded
from the UT build.
- Build: `BUILD_TYPE=ASAN ./build.sh --be` passed.
- Static and format checks: BE build hygiene, clang-format,
check-format, and changed-line clang-tidy passed.
- Regression test: Not run; this changes BE diagnostic log text, not SQL
or HTTP responses.
- Behavior changed: Yes (diagnostic log text only).
- Does this need documentation: No.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
