azure-sdk-for-cpp icon indicating copy to clipboard operation
azure-sdk-for-cpp copied to clipboard

Add `"*"` support for log sanitizer

Open antkmsft opened this issue 3 years ago • 1 comments

.NET supports it, Java does not. Do we want to have it?

Also, we may want to check, if .NET supports it only for HTTP headers, or for the query params as well. Generally, if it is fine in one place, it should be ok in another.

Reasons against: security risks, leaking of Authentication headers. Once we add it, it is a breaking change to remove the support.

antkmsft avatar Jun 03 '22 22:06 antkmsft

PR #3735 has this feature implemented, but do we want it? If yes, the PR can be resurrected.

antkmsft avatar Jun 15 '22 01:06 antkmsft

Hi @antkmsft, we deeply appreciate your input into this project. Regrettably, this issue has remained unresolved for over 2 years and inactive for 30 days, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

github-actions[bot] avatar Jun 03 '24 18:06 github-actions[bot]