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

Configurable application logging to toggle PII on/off for testing #36

Closed
iamkelllly opened this issue Nov 6, 2021 · 1 comment · Fixed by #68
Closed

Configurable application logging to toggle PII on/off for testing #36

iamkelllly opened this issue Nov 6, 2021 · 1 comment · Fixed by #68
Assignees
@eastandwestwind
Labels
enhancement New feature or request
Milestone
Fidesops 1.1.0

Comments

@iamkelllly
Copy link
Contributor

Add additional logging configurations that would allow PII to be displayed for testing purposes.

Context: https://github.com/ethyca/solon/pull/143#discussion_r734639625
@iamkelllly iamkelllly added the enhancement New feature or request label Nov 6, 2021
@iamkelllly
Copy link
Contributor Author

@stevenbenjamin partially implemented this as part of the original query implementation. Some plugs in API are already there, but it's not wired all the way through to the config

@eastandwestwind eastandwestwind self-assigned this Nov 12, 2021
@eastandwestwind eastandwestwind mentioned this issue Nov 15, 2021
Merged
eastandwestwind pushed a commit that referenced this issue Nov 15, 2021
closes #36- masks params provided in % str formatter in logs, unless …
afebf29 
…set otherwise
@eastandwestwind eastandwestwind linked a pull request Nov 15, 2021 that will close this issue
closes #36- masks params provided in % str formatter in logs, unless set otherwise #68
Merged
@iamkelllly iamkelllly added this to the Fidesops 1.1.0 milestone Nov 17, 2021
stevenbenjamin pushed a commit that referenced this issue Nov 18, 2021
@eastandwestwind
closes #36- masks params provided in % str formatter in logs, unless …
3acd69e 
…set otherwise (#68)

* closes #36- masks params provided in % str formatter in logs, unless set otherwise

* remove extra invalid param

* cr changes

* tests

Co-authored-by: catherinesmith <[email protected]>
sanders41 pushed a commit that referenced this issue Sep 22, 2022
@eastandwestwind
closes #36- masks params provided in % str formatter in logs, unless …
1789bdc 
…set otherwise (#68)

* closes #36- masks params provided in % str formatter in logs, unless set otherwise

* remove extra invalid param

* cr changes

* tests

Co-authored-by: catherinesmith <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@eastandwestwind eastandwestwind

Labels
enhancement New feature or request
Projects
None yet
Milestone
Fidesops 1.1.0
Development

Successfully merging a pull request may close this issue.

closes #36- masks params provided in % str formatter in logs, unless set otherwise ethyca/fidesops
2 participants
@eastandwestwind @iamkelllly