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

Remove Masking of Redis Connection Error [#1271] #1288

Merged
sanders41 merged 2 commits into from
Sep 9, 2022
Merged

Remove Masking of Redis Connection Error [#1271] #1288

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b98ffeb
Remove masking of redis connection error.
pattisdr Sep 9, 2022
b6b8fa4
Update Changelog.
pattisdr Sep 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ The types of changes are:
* Fix download link in privacy center [#1264](https://github.com/ethyca/fidesops/pull/1264)
* Make admin ui work when volumes are mounted [#1266](https://github.com/ethyca/fidesops/pull/1266)
* Fixed typo in enum value [#1280](https://github.com/ethyca/fidesops/pull/1280)
* Remove masking of redis error log [#1288](https://github.com/ethyca/fidesops/pull/1288)

### Security

Expand Down
2 changes: 1 addition & 1 deletion src/fidesops/main.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ def start_webserver() -> None:
try:
get_cache()
except (RedisConnectionError, ResponseError) as e:
logger.error("Connection to cache failed: %s", Pii(str(e)))
logger.error("Connection to cache failed: %s", e)
return

scheduler.start()
Expand Down