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

Removing UserInfo from 'created task' logging #725

Merged

Conversation

nharper285
Copy link
Contributor

@nharper285 nharper285 commented Mar 23, 2021

Summary of the Pull Request

removes PII from logging when a task is created

Info on Pull Request

Removal of UserInfo as a logging param in main.py in the ./tasks/ directory.

Validation Steps Performed

Integration tests.

nharper285 and others added 30 commits November 23, 2020 11:26
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
…t irrespective of timestamp."

This reverts commit cbf9dd3.

Need to create a new PR
@bmc-msft bmc-msft changed the title Removing UserInfo from 'created task' trace logging. Removing UserInfo from 'created task' logging Mar 23, 2021
@ghost
Copy link

ghost commented Mar 23, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 18 hours, a condition that will be fulfilled in about 16 hours 7 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@bmc-msft bmc-msft merged commit 1706a91 into microsoft:main Mar 23, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Apr 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants