This repository was archived by the owner on May 16, 2023. It is now read-only.
Enhance logging when filtering checkins#1326
Merged
FelixRottler merged 4 commits intorelease/2.0from Apr 8, 2021
Merged
Conversation
ioangut
previously approved these changes
Apr 8, 2021
pithumke
suggested changes
Apr 8, 2021
...src/main/java/app/coronawarn/server/services/submission/checkins/EventCheckinDataFilter.java
Outdated
Show resolved
Hide resolved
...src/main/java/app/coronawarn/server/services/submission/checkins/EventCheckinDataFilter.java
Show resolved
Hide resolved
pithumke
approved these changes
Apr 8, 2021
|
Kudos, SonarCloud Quality Gate passed! |
pithumke
added a commit
that referenced
this pull request
Apr 9, 2021
* 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
* 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
* 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist
mvn -P integration-tests clean verifyruns for the whole project and, if you touched any code for a service in theservicesfolder, ensure it can be run withspring-boot:run