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

Don't mask redis connection error logs #1271

Closed
pattisdr opened this issue Sep 7, 2022 · 0 comments · Fixed by #1288
Closed

Don't mask redis connection error logs #1271

pattisdr opened this issue Sep 7, 2022 · 0 comments · Fixed by #1288
Assignees
@pattisdr
Labels
enhancement New feature or request

Comments

@pattisdr
Copy link
Contributor

pattisdr commented Sep 7, 2022 •
edited by mfbrown
Loading

Don't mask the error message in the following log, this is useful for troubleshooting

logger.error("Connection to cache failed: %s", Pii(str(e)))
@pattisdr pattisdr added the enhancement New feature or request label Sep 7, 2022
@pattisdr pattisdr self-assigned this Sep 7, 2022
@pattisdr pattisdr mentioned this issue Sep 9, 2022
Merged
10 tasks
sanders41 pushed a commit that referenced this issue Sep 9, 2022
@pattisdr
Remove Masking of Redis Connection Error [#1271] (#1288)
8d0854a 
* Remove masking of redis connection error.

* Update Changelog.
sanders41 pushed a commit that referenced this issue Sep 22, 2022
@pattisdr
Remove Masking of Redis Connection Error [#1271] (#1288)
9edf22c 
* Remove masking of redis connection error.

* Update Changelog.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pattisdr pattisdr

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Masking of Redis Connection Error [#1271] ethyca/fidesops
1 participant
@pattisdr