Skip to content
This repository was archived by the owner on May 16, 2023. It is now read-only.

Sort TraceTimeIntervalWarnings via TraceLocationIDs…#1317

Merged
roesslerj merged 3 commits intorelease/2.0from
feature/deterministic-twp-exportfile
Apr 7, 2021
Merged

Sort TraceTimeIntervalWarnings via TraceLocationIDs…#1317
roesslerj merged 3 commits intorelease/2.0from
feature/deterministic-twp-exportfile

Conversation

@roesslerj
Copy link
Contributor

@roesslerj roesslerj commented Apr 7, 2021 •
edited
Loading

… as this is unambiguous and guaranteed to be unique

Good catch from Pit, related to #1307

NOTE: While I edited this code to resemble TEKs (

cwa-server/services/distribution/src/main/java/app/coronawarn/server/services/distribution/assembly/diagnosiskeys/structure/file/TemporaryExposureKeyExportFile.java

Line 40 in d398d0d

.sorted(new TemporaryExposureKeyComparator())
), I noticed some problems there also: Either we need a Set in between to remove duplicates, than we need the same for TWPs, or we don't...

@roesslerj roesslerj requested a review from pithumke April 7, 2021 06:43
@roesslerj roesslerj force-pushed the feature/deterministic-twp-exportfile branch from 1303493 to 4256888 Compare April 7, 2021 06:49
@roesslerj roesslerj changed the title Sort TraceTimeIntervalWarnings via database ID… Sort TraceTimeIntervalWarnings via TraceLocationIDs… Apr 7, 2021
@roesslerj roesslerj force-pushed the feature/deterministic-twp-exportfile branch from 4256888 to 40d8a71 Compare April 7, 2021 07:04
@roesslerj roesslerj marked this pull request as ready for review April 7, 2021 07:40
@roesslerj roesslerj requested review from a team, EugenM-SAP and ioangut April 7, 2021 07:40
@roesslerj roesslerj enabled auto-merge (squash) April 7, 2021 07:40
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 7, 2021

Kudos, SonarCloud Quality Gate passed!

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

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@roesslerj roesslerj merged commit fb79f1f into release/2.0 Apr 7, 2021
@roesslerj roesslerj deleted the feature/deterministic-twp-exportfile branch April 7, 2021 13:49
pithumke added a commit that referenced this pull request Apr 9, 2021
@ioangut @hilmarf
@FelixRottler @mlenkeit @roesslerj @EugenM-SAP @pithumke
Merge release2 into main (#1319)
6a97bad 
* release 2.0

* Sql permission scrpit distribution (#1296)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1293)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler

* pt-android-poster-1.0.0.pdf

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Fix/add checksum to empty hourly files (#1297)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* feature: use index file with checksum for empty hourly packages, adjusted testing to capture the creation if "middle" hourly packages betwen oldest and newest

* Update services/distribution/src/test/java/app/coronawarn/server/services/distribution/assembly/component/TraceTimeIntervalWarningsStructureProviderTest.java

* isNotDsStore

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Presence tracing parameters (#1301)

* Add presence tracing parameters files and adjust the app config code to include the new configured parameters

* Add Presence tracing parameters mapping tests

* Add environment parameters in the application.yaml tests

* add Event Registration missing template environment variable

* Update application.yaml for tests

* Revert last two commits adding template env variable

* adjust plausible deniability config

this now equals equals the regular size of one check in

* support base32 and base64 in qr codes

* Refactor the Plausible Deniability Parameters structure in Yaml

* Update DeserializedPlausibleDeniabilityParametersTest.java

* Update application yaml for tests

Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>

* Create abstract layer for traceTimeIntervalWarningBundler (#1304)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* update qr code poster parameters (#1311)

Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>

* Feature/sort twps (#1307)

* Simplify code

* Refactor method name to reflect that result is not a single entity, but a collection

* This is already implemented, so we don't need to sort results...

* Add test that displays intended behaviour as already implemented

* Simply remove the TEK headers from the twp file (#1308)

* Add missing trl mapping for Android V2 and IOS app_config (#1312)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* allow optional slash in event registration qr code (#1309)

* Add "cwa-empty-pkg" header to all files (#1310)

* Add "cwa-empty-pkg" header

* Add "cwa-empty-pkg" header to S3Client

* Format code

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Random checkin data padding during submission (#1302)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* Create V13__createPermissionsForEventRegistrationDistribution.sql (#1295)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add random checkins multiplier config

* Implement the fake checkins generation flow at the service level

* Generate fake checkins during submission based on configurable
multiplier

* Resolve security vulnerability and add test coverage

* Sonar fix

* Switch implementations for interval generation

* fix: use hash for inserting trace time intervals

* PR feedback changes

* invalidCheckinData --> validCheckinData

* fix: add additional test for contract between server and client regarding hashing

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* QR Poster Template proto change and code adaptation (#1314)

* fix config (partial)

* proto changes and code adaptations

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* fix: use hex decoding in config because pepper is prob. a hexadecimal string (#1315)

* Feature/exclude current hour twp (#1313)

* Improve test readability

* Adapt test to check whether current hour is NOT contained, but newest not current hour is

* Use JUnit 5 throughout

* Update .mvn/maven.config

* Sort TraceTimeIntervalWarnings via TraceLocationIDs… (#1317)

* Sort TraceTimeIntervalWarnings via TraceLocationId, as this is unambiguous and guaranteed to be unique

* Either we need a set in between to remove duplicates, than we need the same for TWPs, or we don't...

* lower threshold for high-risk from check-ins (#1320)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Feature/twp demo include current hour (#1318)

* Fix copy and paste errors: rename vars `diagnosisKeys` to `warnings`

* Add test that checks that the demo bundler includes the current hour

* Fix/set period when exporting tracewarnings (#1321)

* fix: set period when exporting trace warnings

* fix: float to int after config change

* fix: checkstyle

* Update services/distribution/src/main/java/app/coronawarn/server/services/distribution/assembly/component/QrCodePosterTemplateStructureProvider.java

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

* Add missing submission user permission (#1322)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Revert "Add "cwa-empty-pkg" header to all files (#1310)" (#1323)

This reverts commit 15f6b47.

* Fix checkin interval checks (#1324)

* Adapt tests to allow for start == end time interval

* Allow start time interval to be less than or equal to end time interval

* Feature/return headers submission (#1325)

* REturn checkin information with headers on submission

* Add submission test for new headers cwa-filtered-checkins and cwa-saved-checkins

* Update SubmissionController.java

Co-authored-by: Eugen M <eugen.madean@sap.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Enhance logging when filtering checkins (#1326)

* Additional logging statements in checkin filtering

* fix Checkstyle errors

* Removed unused import

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: ioangut <ioan.gut@sap.com>

* fix: s3 integration tests

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>
Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>
Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>
Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Eugen M <eugen.madean@sap.com>
hilmarf added a commit that referenced this pull request Apr 15, 2021
@alexmoroianu @hilmarf
@ioangut @FelixRottler @mlenkeit @roesslerj @EugenM-SAP @pithumke @Corona-Warn-App-Technical-User @moroianualex
Merge release/2.0 into main (#1338)
9e3b099 
* release 2.0

* Sql permission scrpit distribution (#1296)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1293)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler

* pt-android-poster-1.0.0.pdf

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Fix/add checksum to empty hourly files (#1297)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* feature: use index file with checksum for empty hourly packages, adjusted testing to capture the creation if "middle" hourly packages betwen oldest and newest

* Update services/distribution/src/test/java/app/coronawarn/server/services/distribution/assembly/component/TraceTimeIntervalWarningsStructureProviderTest.java

* isNotDsStore

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Presence tracing parameters (#1301)

* Add presence tracing parameters files and adjust the app config code to include the new configured parameters

* Add Presence tracing parameters mapping tests

* Add environment parameters in the application.yaml tests

* add Event Registration missing template environment variable

* Update application.yaml for tests

* Revert last two commits adding template env variable

* adjust plausible deniability config

this now equals equals the regular size of one check in

* support base32 and base64 in qr codes

* Refactor the Plausible Deniability Parameters structure in Yaml

* Update DeserializedPlausibleDeniabilityParametersTest.java

* Update application yaml for tests

Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>

* Create abstract layer for traceTimeIntervalWarningBundler (#1304)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* update qr code poster parameters (#1311)

Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>

* Feature/sort twps (#1307)

* Simplify code

* Refactor method name to reflect that result is not a single entity, but a collection

* This is already implemented, so we don't need to sort results...

* Add test that displays intended behaviour as already implemented

* Simply remove the TEK headers from the twp file (#1308)

* Add missing trl mapping for Android V2 and IOS app_config (#1312)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* allow optional slash in event registration qr code (#1309)

* Add "cwa-empty-pkg" header to all files (#1310)

* Add "cwa-empty-pkg" header

* Add "cwa-empty-pkg" header to S3Client

* Format code

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Random checkin data padding during submission (#1302)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* Create V13__createPermissionsForEventRegistrationDistribution.sql (#1295)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add random checkins multiplier config

* Implement the fake checkins generation flow at the service level

* Generate fake checkins during submission based on configurable
multiplier

* Resolve security vulnerability and add test coverage

* Sonar fix

* Switch implementations for interval generation

* fix: use hash for inserting trace time intervals

* PR feedback changes

* invalidCheckinData --> validCheckinData

* fix: add additional test for contract between server and client regarding hashing

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* QR Poster Template proto change and code adaptation (#1314)

* fix config (partial)

* proto changes and code adaptations

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* fix: use hex decoding in config because pepper is prob. a hexadecimal string (#1315)

* Feature/exclude current hour twp (#1313)

* Improve test readability

* Adapt test to check whether current hour is NOT contained, but newest not current hour is

* Use JUnit 5 throughout

* Sort TraceTimeIntervalWarnings via TraceLocationIDs… (#1317)

* Sort TraceTimeIntervalWarnings via TraceLocationId, as this is unambiguous and guaranteed to be unique

* Either we need a set in between to remove duplicates, than we need the same for TWPs, or we don't...

* lower threshold for high-risk from check-ins (#1320)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Feature/twp demo include current hour (#1318)

* Fix copy and paste errors: rename vars `diagnosisKeys` to `warnings`

* Add test that checks that the demo bundler includes the current hour

* Fix/set period when exporting tracewarnings (#1321)

* fix: set period when exporting trace warnings

* fix: float to int after config change

* fix: checkstyle

* Update services/distribution/src/main/java/app/coronawarn/server/services/distribution/assembly/component/QrCodePosterTemplateStructureProvider.java

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

* Add missing submission user permission (#1322)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Revert "Add "cwa-empty-pkg" header to all files (#1310)" (#1323)

This reverts commit 15f6b47.

* Fix checkin interval checks (#1324)

* Adapt tests to allow for start == end time interval

* Allow start time interval to be less than or equal to end time interval

* Feature/return headers submission (#1325)

* REturn checkin information with headers on submission

* Add submission test for new headers cwa-filtered-checkins and cwa-saved-checkins

* Update SubmissionController.java

Co-authored-by: Eugen M <eugen.madean@sap.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Enhance logging when filtering checkins (#1326)

* Additional logging statements in checkin filtering

* fix Checkstyle errors

* Removed unused import

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: ioangut <ioan.gut@sap.com>

* Update from https://github.com/corona-warn-app/cwa-protocol-buffers/commit/4f584dacafbe6c57a9ba21919ea5f25c6a84cb4e (#1329)

* Config Updates for PT (#1330)

* pt-config: drop base32 qr codes from allowlist

* pt-config: adjust upper bound for check-in duration intervals

* Add address text box for IOS into app config (#1332)

* Add address text box for IOS into app config

* Update QrCodePosterTemplateStructureProvider.java

* Validate Check-in Plausibility: group check-ins by date. (#1333)

* draft skeleton

* Add env variable for maximum number allowed for checkins per day

* my idea

* will be delivered as HF 2.0.1

* use environment variable for checking the max number of checkins allowed

* .setWidth(xyz.getWidth())

* MAX_ALLOWED_CHECKINS_PER_DAY:20

* Add test for number of checkins

Co-authored-by: ioangut <ioan.gut@sap.com>

* align risk calculation config with latest optimization results (#1336)

* Update maven.config

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>
Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>
Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>
Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Eugen M <eugen.madean@sap.com>
Co-authored-by: ioangut <ioan.gut@sap.com>
Co-authored-by: CWA Technical User <67319337+Corona-Warn-App-Technical-User@users.noreply.github.com>
Co-authored-by: Moroianu Alexandru <moroianu.alexandru@gmail.com>
hilmarf added a commit that referenced this pull request Apr 19, 2021
@ioangut @hilmarf
@FelixRottler @mlenkeit @roesslerj @EugenM-SAP @pithumke @Corona-Warn-App-Technical-User
Release2intomain (#1347)
ebcf437 
* release 2.0

* Sql permission scrpit distribution (#1296)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1293)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler

* pt-android-poster-1.0.0.pdf

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Fix/add checksum to empty hourly files (#1297)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* feature: use index file with checksum for empty hourly packages, adjusted testing to capture the creation if "middle" hourly packages betwen oldest and newest

* Update services/distribution/src/test/java/app/coronawarn/server/services/distribution/assembly/component/TraceTimeIntervalWarningsStructureProviderTest.java

* isNotDsStore

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>

* Presence tracing parameters (#1301)

* Add presence tracing parameters files and adjust the app config code to include the new configured parameters

* Add Presence tracing parameters mapping tests

* Add environment parameters in the application.yaml tests

* add Event Registration missing template environment variable

* Update application.yaml for tests

* Revert last two commits adding template env variable

* adjust plausible deniability config

this now equals equals the regular size of one check in

* support base32 and base64 in qr codes

* Refactor the Plausible Deniability Parameters structure in Yaml

* Update DeserializedPlausibleDeniabilityParametersTest.java

* Update application yaml for tests

Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>

* Create abstract layer for traceTimeIntervalWarningBundler (#1304)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* update qr code poster parameters (#1311)

Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>

* Feature/sort twps (#1307)

* Simplify code

* Refactor method name to reflect that result is not a single entity, but a collection

* This is already implemented, so we don't need to sort results...

* Add test that displays intended behaviour as already implemented

* Simply remove the TEK headers from the twp file (#1308)

* Add missing trl mapping for Android V2 and IOS app_config (#1312)

Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>

* allow optional slash in event registration qr code (#1309)

* Add "cwa-empty-pkg" header to all files (#1310)

* Add "cwa-empty-pkg" header

* Add "cwa-empty-pkg" header to S3Client

* Format code

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Random checkin data padding during submission (#1302)

* Add @component annotation for TraceTimeIntervalWarningsPackageBundler (#1292)

* Create V13__createPermissionsForEventRegistrationDistribution.sql (#1295)

* Create V13__createPermissionsForEventRegistrationDistribution.sql

* Update V13__createPermissionsForEventRegistrationDistribution.sql

* Add random checkins multiplier config

* Implement the fake checkins generation flow at the service level

* Generate fake checkins during submission based on configurable
multiplier

* Resolve security vulnerability and add test coverage

* Sonar fix

* Switch implementations for interval generation

* fix: use hash for inserting trace time intervals

* PR feedback changes

* invalidCheckinData --> validCheckinData

* fix: add additional test for contract between server and client regarding hashing

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* QR Poster Template proto change and code adaptation (#1314)

* fix config (partial)

* proto changes and code adaptations

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* fix: use hex decoding in config because pepper is prob. a hexadecimal string (#1315)

* Feature/exclude current hour twp (#1313)

* Improve test readability

* Adapt test to check whether current hour is NOT contained, but newest not current hour is

* Use JUnit 5 throughout

* Sort TraceTimeIntervalWarnings via TraceLocationIDs… (#1317)

* Sort TraceTimeIntervalWarnings via TraceLocationId, as this is unambiguous and guaranteed to be unique

* Either we need a set in between to remove duplicates, than we need the same for TWPs, or we don't...

* lower threshold for high-risk from check-ins (#1320)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Feature/twp demo include current hour (#1318)

* Fix copy and paste errors: rename vars `diagnosisKeys` to `warnings`

* Add test that checks that the demo bundler includes the current hour

* Fix/set period when exporting tracewarnings (#1321)

* fix: set period when exporting trace warnings

* fix: float to int after config change

* fix: checkstyle

* Update services/distribution/src/main/java/app/coronawarn/server/services/distribution/assembly/component/QrCodePosterTemplateStructureProvider.java

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>

* Add missing submission user permission (#1322)

Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Revert "Add "cwa-empty-pkg" header to all files (#1310)" (#1323)

This reverts commit 15f6b47.

* Fix checkin interval checks (#1324)

* Adapt tests to allow for start == end time interval

* Allow start time interval to be less than or equal to end time interval

* Feature/return headers submission (#1325)

* REturn checkin information with headers on submission

* Add submission test for new headers cwa-filtered-checkins and cwa-saved-checkins

* Update SubmissionController.java

Co-authored-by: Eugen M <eugen.madean@sap.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>

* Enhance logging when filtering checkins (#1326)

* Additional logging statements in checkin filtering

* fix Checkstyle errors

* Removed unused import

Co-authored-by: ioangut <67064882+ioangut@users.noreply.github.com>
Co-authored-by: ioangut <ioan.gut@sap.com>

* Update from https://github.com/corona-warn-app/cwa-protocol-buffers/commit/4f584dacafbe6c57a9ba21919ea5f25c6a84cb4e (#1329)

* Config Updates for PT (#1330)

* pt-config: drop base32 qr codes from allowlist

* pt-config: adjust upper bound for check-in duration intervals

* Add address text box for IOS into app config (#1332)

* Add address text box for IOS into app config

* Update QrCodePosterTemplateStructureProvider.java

* Validate Check-in Plausibility: group check-ins by date. (#1333)

* draft skeleton

* Add env variable for maximum number allowed for checkins per day

* my idea

* will be delivered as HF 2.0.1

* use environment variable for checking the max number of checkins allowed

* .setWidth(xyz.getWidth())

* MAX_ALLOWED_CHECKINS_PER_DAY:20

* Add test for number of checkins

Co-authored-by: ioangut <ioan.gut@sap.com>

* align risk calculation config with latest optimization results (#1336)

* Sign QR poster template (#1340)

* Remove redundant signing decorator implementations

* Add test to ensure that the QR code poster template is signed

* Sign QR code poster template

* Add Trace Time Interval Warning Retention Policy (#1342)

* Update documentation

* Add tests for trace time warning retention policy

* Add trace time warning retention policy

* Update TraceTimeIntervalWarningServiceTest.java

Co-authored-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
Co-authored-by: FelixRottler <72494737+FelixRottler@users.noreply.github.com>
Co-authored-by: Maximilian Lenkeit <mlenkeit@users.noreply.github.com>
Co-authored-by: Jeremias Rößler <jeremias.roessler@sap.com>
Co-authored-by: EugenM-SAP <67458405+EugenM-SAP@users.noreply.github.com>
Co-authored-by: Pit Humke <pithumke@users.noreply.github.com>
Co-authored-by: FelixRottler <fero429@gmail.com>
Co-authored-by: Eugen M <eugen.madean@sap.com>
Co-authored-by: CWA Technical User <67319337+Corona-Warn-App-Technical-User@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@pithumke pithumke pithumke approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@roesslerj @pithumke