[Error Log]Chars "DE" are always replaced with "vaccination/certificateCountry"

Avoid duplicates

• Bug is not mentioned in the FAQ
• Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
• Bug is not already reported in another issue

Technical details

• Device name: LG G4 (LGH-815)
• Android version: 6.0
• App version: 2.6.1

Describe the bug

While trying to debug “some things”, I stumbled upon that error log entry:

2021-08-11T02:24:48.604Z V/DefaultExposureDetectio: Running timeout check (now=2021-08-11T02:24:48.603Z): [TrackedExposureDetection(identifier=3463faf6-6546-4f51-b9eb-85d45fa3af13, startedAt=2021-08-10T06:11:08.298Z, result=NO_MATCHES, finishedAt=2021-08-10T06:11:21.824Z, enfVersion=V2_WINDOW_MOvaccination/certificateCountry), TrackedExposureDetection(identifier=5b5b5d8c-353f-430e-8e9a-fea5312d3773, startedAt=2021-08-10T10:14:46.817Z, result=NO_MATCHES, finishedAt=2021-08-10T10:15:02.430Z, enfVersion=V2_WINDOW_MOvaccination/certificateCountry), TrackedExposureDetection(identifier=6ce6ea79-3fe6-44c8-b41d-1369ab9bf7de, startedAt=2021-08-10T15:17:58.451Z, result=NO_MATCHES, finishedAt=2021-08-10T15:18:13.911Z, enfVersion=V2_WINDOW_MOvaccination/certificateCountry), TrackedExposureDetection(identifier=0927fc48-7831-45a9-a48d-f19ba643841a, startedAt=2021-08-10T19:21:52.010Z, result=NO_MATCHES, finishedAt=2021-08-10T19:22:04.288Z, enfVersion=V2_WINDOW_MOvaccination/certificateCountry), TrackedExposureDetection(identifier=b6702fbe-22ce-430f-ad9f-0e05651d9243, startedAt=2021-08-10T23:24:45.592Z, result=NO_MATCHES, finishedAt=2021-08-10T23:25:01.560Z, enfVersion=V2_WINDOW_MOvaccination/certificateCountry)]

The string that should be “V2_WINDOW_MODE” was “V2_WINDOW_MOvaccination/certificateCountry”. When searching for “vaccination/certificateCountry” in the error log, I found other places, where a “DE” was substituted with “vaccination/certificateCountry”. So, obviously it’s replaced everywhere, where it shouldn’t.

Edit: it only seems to affect uppercase “DE”. Lowercase “de” does not seem to be replaced.

Steps to reproduce the issue

1. Start an error log.
2. Search for “vaccination/certificateCountry”
3. See error

Expected behaviour

“DE” shouldn’t be replaced everywhere.

Possible Fix

Check censor?

Edit: this issue has probably been introduced in 2.3.x, the replaced string has changed in 2.5.1: https://github.com/corona-warn-app/cwa-app-android/blob/2943eff83e919812f9dffd0df2d6644898fa59e1/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/dcc/DccQrCodeCensor.kt#L119-L122

So, it looks like the censor is not only replacing for certificate data here, but everywhere. This issue likely affects other censor replacements for other modules also, when the censor is in general not specific for a module.

Additional context

/CC: @mtwalli

---

Internal Tracking-ID: EXPOSUREAPP-9075

[vaubaehn]

Hi @MikeMcC399 , thanks for checking! ❤️

[MikeMcC399]

@vaubaehn I couldn’t find your exact string when I checked with the 2.9.0 release, however I found several others with V2_WINDOW_MODE, including the following:

2021-09-10T08:29:16.412Z V/TracingStateProvider: latestSubmission: TrackedExposureDetection(identifier=53f1d942-75df-49a6-98c2-229fcae5c918, startedAt=2021-09-10T08:28:17.865Z, result=NO_MATCHES, finishedAt=2021-09-10T08:28:24.212Z, enfVersion=V2_WINDOW_MODE)

where MODE in enfVersion=V2_WINDOW_MODE is now shown correctly. So all is good now with this release! 👍🏻