Xuanwo commented on PR #6603:
URL: https://github.com/apache/opendal/pull/6603#issuecomment-3349952978

   > My point was that later it may be easy to make the mistake of thinking you 
are handling a case in `with_error_response_context` that applies to every 
service, but this would not be true, and you wouldn't know without vigilant 
code reading.
   
   Thank you for this suggestion! Ideally, sensitive information is handled by 
`reqsign`, which inserts the header value marked as `sensitive` so it won’t be 
output in debug logs. I’m considering introducing similar logic for queries as 
well. For now, we’ll fix it for Azure Storage services, which directly include 
tokens in the query string.


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

Reply via email to