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

Censor additional contact journal data on debug logs (EXPOSUREAPP-5035) #2551

Merged
merged 4 commits into from
Mar 9, 2021

Conversation

d4rken
Copy link
Member

@d4rken d4rken commented Mar 8, 2021

  • Phone/Email for location and person
  • Circumstances for encounter and visit

To make testing easier I've removed the differentiation between tester and non tester builds as there has not yet been a case AFAIK, where censoring this data in tester builds was an issue.

@d4rken d4rken added enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers labels Mar 8, 2021
@d4rken d4rken added this to the 1.15.0 milestone Mar 8, 2021
@d4rken d4rken requested a review from a team March 8, 2021 14:52
Base automatically changed from feature/4451-technical-support-main to release/1.15.x March 8, 2021 16:09
@AlexanderAlferov AlexanderAlferov self-assigned this Mar 9, 2021
# Conflicts:
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/BugReportingSharedModule.kt
@ralfgehrer ralfgehrer added the prio PRs to review first. label Mar 9, 2021
@ralfgehrer ralfgehrer self-assigned this Mar 9, 2021
@sonarcloud
Copy link

sonarcloud bot commented Mar 9, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

92.2% 92.2% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@ralfgehrer ralfgehrer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested on Pixel (Android 10),

@ralfgehrer ralfgehrer merged commit 5321f2d into release/1.15.x Mar 9, 2021
@ralfgehrer ralfgehrer deleted the feature/5035-contactdiary-censoring branch March 9, 2021 17:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers prio PRs to review first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants