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

Commit

Permalink
Skip Masking of Uvicorn Logs [#766] (#831)
Browse files Browse the repository at this point in the history
* Update get_fides_log_record_factory to skip masking of all uvicorn logs.

* Update changelog.
  • Loading branch information
pattisdr authored Jul 7, 2022
1 parent 02076d6 commit d05d8f9
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ The types of changes are:
* Backend UI deployment [#827](https://github.com/ethyca/fidesops/pull/827)
* Fix publish_docs CI action [#818](https://github.com/ethyca/fidesops/pull/818)
* Bump fideslib to handle base64 encoded password [#820](https://github.com/ethyca/fidesops/pull/820)
* Stop masking uvicorn logs by default [#831](https://github.com/ethyca/fidesops/pull/831)

## [1.6.1](https://github.com/ethyca/fidesops/compare/1.6.0...1.6.1)

Expand Down
2 changes: 1 addition & 1 deletion src/fidesops/util/logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ def factory( # pylint: disable=R0913
) -> logging.LogRecord:
env_log_pii: bool = os.getenv("FIDESOPS__LOG_PII", "").lower() == "true"
new_args = args
if not env_log_pii:
if not env_log_pii and not name.startswith("uvicorn"):
new_args = tuple(_mask_pii_for_logs(arg) for arg in args)
return logging.LogRecord(
name=name,
Expand Down

0 comments on commit d05d8f9

Please sign in to comment.