Skip to content
This repository was archived by the owner on Nov 30, 2022. It is now read-only.

Use RedactedStruct for config#13

Merged
zachmargolis merged 1 commit intomainfrom
margolis-redacted-struct
Mar 25, 2021
Merged

Use RedactedStruct for config#13
zachmargolis merged 1 commit intomainfrom
margolis-redacted-struct

Conversation

@zachmargolis
Copy link
Contributor

Why: To minimize the chances we accidentally log sensitive credentials

**Why**: To minimize the chances we accidentally log sensitive credentials
Copy link

@stevegsa stevegsa left a comment

Choose a reason for hiding this comment

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

LGTM

@zachmargolis zachmargolis merged commit 6e1c9aa into main Mar 25, 2021
@zachmargolis zachmargolis deleted the margolis-redacted-struct branch March 25, 2021 15:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants