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.

Sign up for GitHub

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

Jump to bottom

fix: Remove API key logging #606

Merged

TylerHelmuth merged 1 commit into main from tyler.update-logging

Feb 3, 2023

Contributor

TylerHelmuth commented Feb 3, 2023

Which problem is this PR solving?

A customer noticed that one of our debug log includes the plaintext API key

Short description of the changes

removes an api key from a debug log


          Remove API key logging

8225ca7

TylerHelmuth requested a review from a team as a code owner

February 3, 2023 20:30

kentquirk approved these changes

View reviewed changes

TylerHelmuth merged commit 37c1f01 into main

TylerHelmuth deleted the tyler.update-logging branch

February 3, 2023 22:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet