goutamadwant opened a new pull request, #25927:
URL: https://github.com/apache/datafusion/pull/25927

   ## Which issue does this PR close?
   
   - Closes #25632.
   
   ## Rationale for this change
   
   Object-store URLs can contain credentials, ABFS container identifiers, and 
query tokens. Debug output and missing-store errors currently expose parts of 
those URLs in diagnostics.
   
   ## What changes are included in this PR?
   
   - Show only the scheme, host, and port in `ObjectStoreUrl` and registry 
Debug output and in missing-store errors.
   - Keep URL parsing, display, registry lookup keys, and serialized scan 
configurations unchanged.
   - Add regression tests for credential redaction, ABFS/ABFSS URLs, registry 
behavior, and protobuf round-trip preservation.
   
   ## What is the testing strategy for this PR?
   
   - `cargo fmt --all -- --check`
   - `cargo clippy --all-targets --all-features -- -D warnings`
   - `cargo test -p datafusion-execution` (124 unit tests and 19 doc tests 
passed)
   - `cargo test -p datafusion-proto --test proto_integration` (266 tests 
passed)
   
   ## Are there any user-facing changes?
   
   Debug output and missing-store error messages now omit URL userinfo, paths, 
queries, and fragments. Object-store lookup and serialized URL behavior are 
unchanged.


-- 
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]

Reply via email to