Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't redact Key Vault header values in logs #12077

Merged
merged 2 commits into from
Jun 22, 2020
Merged

Conversation

chlowell
Copy link
Member

Key Vault has a couple custom headers whose values we've been unwittingly redacting from logs. This configures HttpLoggingPolicy to log their values.

@chlowell chlowell added KeyVault Client This issue points to a problem in the data-plane of the library. labels Jun 16, 2020
@chlowell chlowell requested a review from iscai-msft June 16, 2020 20:17
@chlowell chlowell requested a review from schaabs as a code owner June 16, 2020 20:17
iscai-msft
iscai-msft previously approved these changes Jun 17, 2020
Copy link
Contributor

@iscai-msft iscai-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@chlowell chlowell merged commit 2c8bfa1 into Azure:master Jun 22, 2020
@chlowell chlowell deleted the kv-headers branch June 22, 2020 21:19
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Jun 23, 2020
…into regenerate_certs

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  Resolve Pip Related Errors (Azure#12157)
  [Service Bus] Delete swagger file (Azure#12097)
  Add set_delete_after_tag as a parameter on ResourceGroupPreparer (Azure#11749)
  Custom Vision Training: release_3_3 Prediction: release_3_1 (Azure#12105)
  Don't redact Key Vault header values in logs (Azure#12077)
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Jun 23, 2020
…into regenerate_keys

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  Resolve Pip Related Errors (Azure#12157)
  [Service Bus] Delete swagger file (Azure#12097)
  Add set_delete_after_tag as a parameter on ResourceGroupPreparer (Azure#11749)
  Custom Vision Training: release_3_3 Prediction: release_3_1 (Azure#12105)
  Don't redact Key Vault header values in logs (Azure#12077)
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants