Engagement/jessica/15406 cli download report #15675

JessicaWNava · 2024-08-23T21:59:53Z

This PR adds a CLI command to download a report from a specified environment while optionally (unless it is prod) removing PII.

Test Steps:

Get a report id to query for, make sure it is returned with and without the PII removed as you specify in the command
Get a report id that is for an HL7 report and make sure it returns a message saying it is the wrong type.
Try to download a prod report with remove-pii set to false and make sure it does not allow this.
Example query: downloadMessage -o /Users/jessicawillis/git/newnew/prime-reportstream/prime-router/src/testIntegration/resources/datatests/FHIR_to_HL7/downloaded.fhir --env local -r 777fc1d7-dd51-4f01-ad56-6e51b4721caa --remove-pii false

Changes

CLI command added to download a report

Checklist

Testing

Tested locally?
Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
(For Changes to /frontend-react/...) Ran npm run lint:write?
Added tests?

Process

Are there licensing issues with any new dependencies introduced?
Includes a summary of what a code reviewer should test/verify?
Updated the release notes?
Database changes are submitted as a separate PR?
DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue

To Be Done

Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?
- new endpoint A
- new endpoint B
Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated
- Spoofing (faking authenticity)
  - Threat T, which could be achieved by A, is mitigated by M
- Tampering (influence or sabotage the integrity of information, data, or system)
- Repudiation (the ability to dispute the origin or originator of an action)
- Information disclosure (data made available to entities who should not have it)
- Denial of service (make a resource unavailable)
- Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)
Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

… types of data

…tion.caristix.com/v2/HL7v2.5/Tables/0105

This reverts commit 8abde73.

…nd optionally (unless it is prod) removes PII

github-actions · 2024-08-23T22:00:11Z

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Manifest Files

…https://github.com/CDCgov/prime-reportstream into engagement/jessica/15405-deidentification-enrichment

github-actions · 2024-08-23T22:12:32Z

Test Results

1 217 tests +5 1 213 ✅ +5 6m 39s ⏱️ -1s
158 suites ±0 4 💤 ±0
158 files ±0 0 ❌ ±0

Results for commit a97f505. ± Comparison against base commit a318ff9.

♻️ This comment has been updated with latest results.

github-actions · 2024-08-23T22:33:29Z

Integration Test Results

53 files ±0 53 suites ±0 28m 12s ⏱️ +9s
408 tests ±0 399 ✅ ±0 9 💤 ±0 0 ❌ ±0
411 runs ±0 402 ✅ ±0 9 💤 ±0 0 ❌ ±0

Results for commit a97f505. ± Comparison against base commit a318ff9.

♻️ This comment has been updated with latest results.

victor-chaparro · 2024-09-04T00:25:42Z

prime-router/src/main/kotlin/azure/ReportFunction.kt

+        @HttpTrigger(
+            name = "downloadReport",
+            methods = [HttpMethod.GET],
+            authLevel = AuthorizationLevel.ANONYMOUS,


I think it would be better to use an azure function key to call this API, since we will have to manually call this API every time we need a report and we won't be using the front end to call this API. It will make it easier to use and we wouldn't have to get a bearer token every time we wanted to use it.
There's an example inside ReportFunction.kt in line 70. You'd just have to set to authLevel to authLevel = AuthorizationLevel.FUNCTION and once it's in staging and prod we can generate a random key for it.
If we do it that way we can remove the authentication logic and just rely on azure's authorization.

sonarcloud · 2024-09-06T20:03:39Z

Quality Gate failed

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

* fix copy * update PDF * update timestamp * testing re enabling file download * add some logging to e2e test run * increase timeout * 15607 remove non catchall hl7 to fhir mappings (#15733) * test uploading trace * Generating a version.json file with the current short commit id and pu… (#15789) * Generating a version.json file with the current short commit id and putting it in with the Azure resources * Update up-receiver-migration-validate-test-data.md (#15604) Improvements to the Migration - Validate Test Data template: - Updated the default labels - Added space for STLT name - Added Slack post reminder in the Acceptance criteria * Oo/oslynn/15664/update rest transport for oracle rln etor nbs results (#15721) * Update REST Transport to allow ApiKey for Oracle-RLN to read from vault * Update REST Transport to allow ApiKey for Oracle-RLN to read from vault * Update REST Transport to allow ApiKey for Oracle-RLN to read from vault * Update REST Transport to allow Oracle-RLN to read apiKey from the vault * Update REST Transport to allow Oracle-RLN to read apiKey from the vault * Fixed RestTransport Integration Test * Wasn't working in Staging or on Arnej's machine, so changed to be generating the file in the aureFunctionsDir (#15802) * Engagement/jessica/15406 cli download report (#15675) * Created an API that downloads files from a specified environment and optionally (unless it is prod) removes PII * slightly cleaner tracing * console added * needed to ignore console * test sort method * simply skip test if mocked * invert truthiness * Bump actions/setup-java in /.github/actions/build-backend (#15817) Bumps [actions/setup-java](https://github.com/actions/setup-java) from 4.2.2 to 4.3.0. - [Release notes](https://github.com/actions/setup-java/releases) - [Commits](actions/setup-java@6a0805f...2dfa201) --- updated-dependencies: - dependency-name: actions/setup-java dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump actions/setup-java from 4.2.2 to 4.3.0 (#15818) Bumps [actions/setup-java](https://github.com/actions/setup-java) from 4.2.2 to 4.3.0. - [Release notes](https://github.com/actions/setup-java/releases) - [Commits](actions/setup-java@6a0805f...2dfa201) --- updated-dependencies: - dependency-name: actions/setup-java dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump actions/setup-java in /.github/actions/build-submissions (#15816) Bumps [actions/setup-java](https://github.com/actions/setup-java) from 4.2.2 to 4.3.0. - [Release notes](https://github.com/actions/setup-java/releases) - [Commits](actions/setup-java@6a0805f...2dfa201) --- updated-dependencies: - dependency-name: actions/setup-java dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump bridgecrewio/checkov-action from 12.2864.0 to 12.2867.0 (#15819) Bumps [bridgecrewio/checkov-action](https://github.com/bridgecrewio/checkov-action) from 12.2864.0 to 12.2867.0. - [Release notes](https://github.com/bridgecrewio/checkov-action/releases) - [Commits](bridgecrewio/checkov-action@c942186...6429623) --- updated-dependencies: - dependency-name: bridgecrewio/checkov-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * update dates * 15262 - Receiver Status smoke tests (#15785) * 15262 - Fixed smoke tests * Created receiver-status-page-user-flow.spec.ts and made tests as sanity checks. Moved spec files into correct folders. * Fixed linting * Fixed path * Fixed path * Removed commented code * Fixed spacing issues * Fixed spacing issues * Fixed spacing issues * Fixed failing tests * Fixed failing tests * Added check if using mock * Added check if using mock * Add Receive step to UP (#15621) * add QueueMessage to shared module as interface * add azure table update to fhir converter * add tracking for no report * add BlobUtils * rename function to parsePrimeRouterQueueMessage * cast to specific queue message in up steps * remove tableAccess * fix fhirconverter tests * fix so headers is deserialized * add flyway script tup update report_file * update flyway script * update TopicConverter to accept null topic * move from convert step to receive function * inital setup for receive step * add event and fhirenginerunresult return * move all processing over to FHIRReceiver * working code * complete unit tests * complete integration tests * remove unnecessary test * fix SubmissionControllerTest * delete files * delete files * fix integration test * change to doReceive * remove unneeded mock * mock action logger error * lint format * print error in test * stacktrace to string * use stackTraceToString() * print stacktrace * pass metadata into report creation * review fixup * changed assert * update from review * fixup based on new plan to parse messages * fix some lint in QueueMessage * add fields to submission table and change name * replace SubmissionsEntity * update receiver integration tests * add elrReceiveQueueName * filterHeaders toMap and serialize message * fix sub controller int test * add azure event checks to integration tests * add documentation to MimeFormat.kt * create TableAccess and update tests * review fixup * remove space * test fixup * add azure event for action logger errors * trackExternalInputReport to blobinfo property * fix tests * update SubmissionTableService * no need to clear mocks * table client * fix integration tests! * gettableclient catch with retry * document * add reset and rename SubmissionEntity * multi threading test win * comment fixup * file from merge * move thread test over to different file * rename val and add asserts for report_file * remove piiRemoved.fhir * Engagement/nj prework (#15529) * Fixed New Jersey's suppress OBX-18 * Added NJ pre-work * Updated receiving facilities. * create documentation for hl7-fhir transforms (#15670) * create documentation for hl7-fhir transforms * update documentation for hl7-fhir transforms * update documentation formatting for hl7-fhir transforms * update documentation for null vs empty * Bump the react-router group across 1 directory with 2 updates (#15627) Bumps the react-router group with 2 updates in the /frontend-react directory: [react-router](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router) and [react-router-dom](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router-dom). Updates `react-router` from 6.25.1 to 6.26.1 - [Release notes](https://github.com/remix-run/react-router/releases) - [Changelog](https://github.com/remix-run/react-router/blob/main/packages/react-router/CHANGELOG.md) - [Commits](https://github.com/remix-run/react-router/commits/react-router@6.26.1/packages/react-router) Updates `react-router-dom` from 6.25.1 to 6.26.1 - [Release notes](https://github.com/remix-run/react-router/releases) - [Changelog](https://github.com/remix-run/react-router/blob/main/packages/react-router-dom/CHANGELOG.md) - [Commits](https://github.com/remix-run/react-router/commits/react-router-dom@6.26.1/packages/react-router-dom) --- updated-dependencies: - dependency-name: react-router dependency-type: direct:production update-type: version-update:semver-minor dependency-group: react-router - dependency-name: react-router-dom dependency-type: direct:production update-type: version-update:semver-minor dependency-group: react-router ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: etanb <etanb@users.noreply.github.com> * Bump lint-staged from 15.2.9 to 15.2.10 in /frontend-react (#15780) Bumps [lint-staged](https://github.com/lint-staged/lint-staged) from 15.2.9 to 15.2.10. - [Release notes](https://github.com/lint-staged/lint-staged/releases) - [Changelog](https://github.com/lint-staged/lint-staged/blob/master/CHANGELOG.md) - [Commits](lint-staged/lint-staged@v15.2.9...v15.2.10) --- updated-dependencies: - dependency-name: lint-staged dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump the applicationinsights group in /frontend-react with 2 updates (#15778) Bumps the applicationinsights group in /frontend-react with 2 updates: [@microsoft/applicationinsights-react-js](https://github.com/microsoft/applicationinsights-react-js) and [@microsoft/applicationinsights-web](https://github.com/microsoft/ApplicationInsights-JS). Updates `@microsoft/applicationinsights-react-js` from 17.3.1 to 17.3.2 - [Release notes](https://github.com/microsoft/applicationinsights-react-js/releases) - [Changelog](https://github.com/microsoft/applicationinsights-react-js/blob/main/RELEASES.md) - [Commits](microsoft/applicationinsights-react-js@17.3.1...17.3.2) Updates `@microsoft/applicationinsights-web` from 3.3.1 to 3.3.2 - [Release notes](https://github.com/microsoft/ApplicationInsights-JS/releases) - [Changelog](https://github.com/microsoft/ApplicationInsights-JS/blob/main/RELEASES.md) - [Commits](https://github.com/microsoft/ApplicationInsights-JS/commits) --- updated-dependencies: - dependency-name: "@microsoft/applicationinsights-react-js" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: applicationinsights - dependency-name: "@microsoft/applicationinsights-web" dependency-type: direct:production update-type: version-update:semver-patch dependency-group: applicationinsights ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @types/sanitize-html from 2.11.0 to 2.13.0 in /frontend-react (#15683) Bumps [@types/sanitize-html](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/sanitize-html) from 2.11.0 to 2.13.0. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/sanitize-html) --- updated-dependencies: - dependency-name: "@types/sanitize-html" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump the testing-library group across 1 directory with 2 updates (#15739) Bumps the testing-library group with 2 updates in the /frontend-react directory: [@testing-library/jest-dom](https://github.com/testing-library/jest-dom) and [@testing-library/react](https://github.com/testing-library/react-testing-library). Updates `@testing-library/jest-dom` from 6.4.8 to 6.5.0 - [Release notes](https://github.com/testing-library/jest-dom/releases) - [Changelog](https://github.com/testing-library/jest-dom/blob/main/CHANGELOG.md) - [Commits](testing-library/jest-dom@v6.4.8...v6.5.0) Updates `@testing-library/react` from 16.0.0 to 16.0.1 - [Release notes](https://github.com/testing-library/react-testing-library/releases) - [Changelog](https://github.com/testing-library/react-testing-library/blob/main/CHANGELOG.md) - [Commits](testing-library/react-testing-library@v16.0.0...v16.0.1) --- updated-dependencies: - dependency-name: "@testing-library/jest-dom" dependency-type: direct:development update-type: version-update:semver-minor dependency-group: testing-library - dependency-name: "@testing-library/react" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: testing-library ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump axios from 1.7.5 to 1.7.7 in /frontend-react (#15741) Bumps [axios](https://github.com/axios/axios) from 1.7.5 to 1.7.7. - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v1.7.5...v1.7.7) --- updated-dependencies: - dependency-name: axios dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump jsdom from 24.1.1 to 25.0.0 in /frontend-react (#15744) Bumps [jsdom](https://github.com/jsdom/jsdom) from 24.1.1 to 25.0.0. - [Release notes](https://github.com/jsdom/jsdom/releases) - [Changelog](https://github.com/jsdom/jsdom/blob/main/Changelog.md) - [Commits](jsdom/jsdom@24.1.1...25.0.0) --- updated-dependencies: - dependency-name: jsdom dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @trussworks/react-uswds from 9.0.0 to 9.1.0 in /frontend-react (#15745) Bumps [@trussworks/react-uswds](https://github.com/trussworks/react-uswds) from 9.0.0 to 9.1.0. - [Release notes](https://github.com/trussworks/react-uswds/releases) - [Changelog](https://github.com/trussworks/react-uswds/blob/main/CHANGELOG.md) - [Commits](trussworks/react-uswds@9.0.0...9.1.0) --- updated-dependencies: - dependency-name: "@trussworks/react-uswds" dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump tsx from 4.17.0 to 4.19.0 in /frontend-react (#15746) Bumps [tsx](https://github.com/privatenumber/tsx) from 4.17.0 to 4.19.0. - [Release notes](https://github.com/privatenumber/tsx/releases) - [Changelog](https://github.com/privatenumber/tsx/blob/master/release.config.cjs) - [Commits](privatenumber/tsx@v4.17.0...v4.19.0) --- updated-dependencies: - dependency-name: tsx dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Experience/15262/receiver status smoke (#15824) * 15262 - Fixed smoke tests * Created receiver-status-page-user-flow.spec.ts and made tests as sanity checks. Moved spec files into correct folders. * Fixed linting * Fixed path * Fixed path * Removed commented code * Fixed spacing issues * Fixed spacing issues * Fixed spacing issues * Fixed failing tests * Fixed failing tests * Added check if using mock * Added check if using mock * Fixing receiver name failing test * Skipped tests until filters have been fixed per #15737 * Bump @types/react from 18.3.3 to 18.3.5 in /frontend-react (#15747) Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.3.3 to 18.3.5. - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react) --- updated-dependencies: - dependency-name: "@types/react" dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump sass from 1.77.8 to 1.78.0 in /frontend-react (#15779) Bumps [sass](https://github.com/sass/dart-sass) from 1.77.8 to 1.78.0. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md) - [Commits](sass/dart-sass@1.77.8...1.78.0) --- updated-dependencies: - dependency-name: sass dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump postcss from 8.4.41 to 8.4.45 in /frontend-react (#15782) Bumps [postcss](https://github.com/postcss/postcss) from 8.4.41 to 8.4.45. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.41...8.4.45) --- updated-dependencies: - dependency-name: postcss dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump the vite group across 1 directory with 2 updates (#15799) Bumps the vite group with 2 updates in the /frontend-react directory: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) and [vite-plugin-checker](https://github.com/fi3ework/vite-plugin-checker). Updates `vite` from 5.3.4 to 5.4.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v5.4.3/packages/vite) Updates `vite-plugin-checker` from 0.7.2 to 0.8.0 - [Release notes](https://github.com/fi3ework/vite-plugin-checker/releases) - [Changelog](https://github.com/fi3ework/vite-plugin-checker/blob/main/CHANGELOG.md) - [Commits](https://github.com/fi3ework/vite-plugin-checker/compare/vite-plugin-checker@0.7.2...vite-plugin-checker@0.8.0) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development update-type: version-update:semver-minor dependency-group: vite - dependency-name: vite-plugin-checker dependency-type: direct:development update-type: version-update:semver-minor dependency-group: vite ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @playwright/test in /frontend-react in the playwright group (#15807) Bumps the playwright group in /frontend-react with 1 update: [@playwright/test](https://github.com/microsoft/playwright). Updates `@playwright/test` from 1.46.1 to 1.47.0 - [Release notes](https://github.com/microsoft/playwright/releases) - [Commits](microsoft/playwright@v1.46.1...v1.47.0) --- updated-dependencies: - dependency-name: "@playwright/test" dependency-type: direct:development update-type: version-update:semver-minor dependency-group: playwright ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * disable smoke tests for receiver status temporarily (#15836) * Engagement/jessica/15541 cli add code to condition (#15803) * Added code to condition to the fhirdata cli tool * added schema that handles converting unwanted conditions to notes acccording to MARS (#15764) * Bump the react-query group across 1 directory with 2 updates (#15838) Bumps the react-query group with 2 updates in the /frontend-react directory: [@tanstack/react-query](https://github.com/TanStack/query/tree/HEAD/packages/react-query) and [@tanstack/react-query-devtools](https://github.com/TanStack/query/tree/HEAD/packages/react-query-devtools). Updates `@tanstack/react-query` from 5.51.23 to 5.55.4 - [Release notes](https://github.com/TanStack/query/releases) - [Commits](https://github.com/TanStack/query/commits/v5.55.4/packages/react-query) Updates `@tanstack/react-query-devtools` from 5.51.23 to 5.55.4 - [Release notes](https://github.com/TanStack/query/releases) - [Commits](https://github.com/TanStack/query/commits/v5.55.4/packages/react-query-devtools) --- updated-dependencies: - dependency-name: "@tanstack/react-query" dependency-type: direct:production update-type: version-update:semver-minor dependency-group: react-query - dependency-name: "@tanstack/react-query-devtools" dependency-type: direct:production update-type: version-update:semver-minor dependency-group: react-query ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joseph Andersen <12385932+jpandersen87@users.noreply.github.com> * Bump express from 4.19.2 to 4.20.0 in /frontend-react (#15837) Bumps [express](https://github.com/expressjs/express) from 4.19.2 to 4.20.0. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.19.2...4.20.0) --- updated-dependencies: - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @okta/okta-signin-widget in /frontend-react in the okta group (#15809) Bumps the okta group in /frontend-react with 1 update: [@okta/okta-signin-widget](https://github.com/okta/okta-signin-widget). Updates `@okta/okta-signin-widget` from 7.21.2 to 7.23.0 - [Release notes](https://github.com/okta/okta-signin-widget/releases) - [Changelog](https://github.com/okta/okta-signin-widget/blob/master/webpack.release.config.js) - [Commits](okta/okta-signin-widget@okta-signin-widget-7.21.2...okta-signin-widget-7.23.0) --- updated-dependencies: - dependency-name: "@okta/okta-signin-widget" dependency-type: direct:production update-type: version-update:semver-minor dependency-group: okta ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump the vitest group in /frontend-react with 3 updates (#15457) Bumps the vitest group in /frontend-react with 3 updates: [@vitest/coverage-istanbul](https://github.com/vitest-dev/vitest/tree/HEAD/packages/coverage-istanbul), [@vitest/ui](https://github.com/vitest-dev/vitest/tree/HEAD/packages/ui) and [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest). Updates `@vitest/coverage-istanbul` from 2.0.4 to 2.0.5 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.5/packages/coverage-istanbul) Updates `@vitest/ui` from 2.0.4 to 2.0.5 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.5/packages/ui) Updates `vitest` from 2.0.4 to 2.0.5 - [Release notes](https://github.com/vitest-dev/vitest/releases) - [Commits](https://github.com/vitest-dev/vitest/commits/v2.0.5/packages/vitest) --- updated-dependencies: - dependency-name: "@vitest/coverage-istanbul" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: vitest - dependency-name: "@vitest/ui" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: vitest - dependency-name: vitest dependency-type: direct:development update-type: version-update:semver-patch dependency-group: vitest ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Joseph Andersen <12385932+jpandersen87@users.noreply.github.com> * Bump bridgecrewio/checkov-action from 12.2867.0 to 12.2869.0 (#15845) Bumps [bridgecrewio/checkov-action](https://github.com/bridgecrewio/checkov-action) from 12.2867.0 to 12.2869.0. - [Release notes](https://github.com/bridgecrewio/checkov-action/releases) - [Commits](bridgecrewio/checkov-action@6429623...08a0f07) --- updated-dependencies: - dependency-name: bridgecrewio/checkov-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Engagement/jessica/15405 cleanup deidentification enrichment (#15821) * Changed where the files used and produced by the remove pii test live as well as delete the output file after the test runs * Engagement/jessica/15540 fhirdata specify receiver (#15800) * Added the ability to specify the receiver settings for the fhirdata command * Made the receiver schema optional if the receiver is specified. * Checking the receiver for their format and using it. if applicable. Applying enrichment schemas across the board if a receiver is specified and has them * Engagement/jessica/15406 download report cleanup (#15847) * Capturing the report not existing error so that we can return a pretty one. Fixing a condition in the pii removal yml. Returning a prettier body for a report * Bump chromaui/action from 11.7.1 to 11.8.0 (#15848) Bumps [chromaui/action](https://github.com/chromaui/action) from 11.7.1 to 11.8.0. - [Release notes](https://github.com/chromaui/action/releases) - [Commits](chromaui/action@b984808...95f238d) --- updated-dependencies: - dependency-name: chromaui/action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * #15217 - Implement smoke test user flow for Last Mile Failures (#15843) * #15217 - Implement smoke test user flow for Last Mile Failures * Fixed linting issues * Fixed linting issues --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: etanb <etan.berkowitz@gmail.com> Co-authored-by: Jack Wang <jack.wang@focusconsulting.io> Co-authored-by: Angela DeGolier <39102333+adegolier@users.noreply.github.com> Co-authored-by: chris-kuryak <117934818+chris-kuryak@users.noreply.github.com> Co-authored-by: OTT SATHNGAM <ott.sathngam@gmail.com> Co-authored-by: JessicaWNava <119880261+JessicaWNava@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: etanb <etanb@users.noreply.github.com> Co-authored-by: Penelope Lischer <102491809+penny-lischer@users.noreply.github.com> Co-authored-by: Brick Green <86254221+brick-green@users.noreply.github.com> Co-authored-by: victor-chaparro <117938212+victor-chaparro@users.noreply.github.com> Co-authored-by: James Gilmore <109554461+GilmoreA6@users.noreply.github.com> Co-authored-by: Joseph Andersen <12385932+jpandersen87@users.noreply.github.com> Co-authored-by: lucero-v <165824244+lucero-v@users.noreply.github.com>

JessicaWNava and others added 18 commits August 6, 2024 15:37

Creating custom FHIR function that returns a random value for various…

9dd54c3

… types of data

Removing logging

2b58146

Addressing PR feedbakc and doing some cleanup after further testing

46bf5c4

Adding in any missing types that were required per https://hl7-defini…

8cd7f3a

…tion.caristix.com/v2/HL7v2.5/Tables/0105

Starting work on using the fake values to remove PII froma bundle

94ca3dc

PII removal commit 2

e07c4f7

pii removal number 3

a890872

Moving PII removal over to code

7064ee6

Removing AOEs and updating condition filter to include RSV

9c06530

Merging changes

1dad44f

Changing to use the transform again instead of code for all but id

51a3fd3

Merge branch 'master' of https://github.com/CDCgov/prime-reportstream

d9a10fe

Merging changes from master

cdf4f2d

Added ability to set properties with index

fc560af

PII removal combo of transform and code

4c9916c

create documentation for hl7-fhir transforms

8abde73

Revert "create documentation for hl7-fhir transforms"

5a6131b

This reverts commit 8abde73.

Created a command that downloads files from a specified environment a…

7bf0da2

…nd optionally (unless it is prod) removes PII

JessicaWNava requested a review from a team as a code owner August 23, 2024 21:59

JessicaWNava assigned oslynn, GilmoreA6, victor-chaparro and lucero-v Aug 23, 2024

JessicaWNava temporarily deployed to staging August 23, 2024 22:00 — with GitHub Actions Inactive

JessicaWNava added 2 commits August 23, 2024 15:01

Removing scratch file

824f95f

Merge branch 'engagement/jessica/15405-deidentification-enrichment' of …

4b2cc1e

…https://github.com/CDCgov/prime-reportstream into engagement/jessica/15405-deidentification-enrichment

Tests added for PII removal

6c332da

JessicaWNava temporarily deployed to staging August 29, 2024 19:01 — with GitHub Actions Inactive

Moving download report over from being A CLI command to being an API

561d63d

JessicaWNava temporarily deployed to staging September 3, 2024 18:27 — with GitHub Actions Inactive

Finished updating the tests

b0656ac

JessicaWNava temporarily deployed to staging September 3, 2024 18:54 — with GitHub Actions Inactive

victor-chaparro reviewed Sep 4, 2024

View reviewed changes

Changing how ReportFunction is initialized in the tests

5f0d760

JessicaWNava temporarily deployed to staging September 4, 2024 13:42 — with GitHub Actions Inactive

Updated how we do security for the endpoint and fixed a test

46fbf8f

JessicaWNava temporarily deployed to staging September 4, 2024 14:14 — with GitHub Actions Inactive

Fixing tests

3d414bb

JessicaWNava temporarily deployed to staging September 4, 2024 16:11 — with GitHub Actions Inactive

Fixing remove pii address

994b97f

JessicaWNava had a problem deploying to staging September 4, 2024 18:03 — with GitHub Actions Error

Fixing a FHIR path

f515b1b

JessicaWNava temporarily deployed to staging September 4, 2024 18:20 — with GitHub Actions Inactive

JessicaWNava added 3 commits September 4, 2024 14:36

Merge branch 'master' of https://github.com/CDCgov/prime-reportstream

e6cebab

Merge branch 'master' of https://github.com/CDCgov/prime-reportstream

aee1237

Merging changes from master

0f782ec

JessicaWNava temporarily deployed to staging September 4, 2024 19:57 — with GitHub Actions Inactive

victor-chaparro approved these changes Sep 6, 2024

View reviewed changes

Fixing function name of an API call accidentally duplicated

9b664ae

JessicaWNava temporarily deployed to staging September 6, 2024 18:47 — with GitHub Actions Inactive

Merge branch 'master' into engagement/jessica/15406-cli-download-report

a97f505

JessicaWNava temporarily deployed to staging September 6, 2024 19:43 — with GitHub Actions Inactive

JessicaWNava enabled auto-merge (squash) September 6, 2024 19:45

JessicaWNava merged commit 1220903 into master Sep 6, 2024
21 of 22 checks passed

JessicaWNava deleted the engagement/jessica/15406-cli-download-report branch September 6, 2024 20:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Engagement/jessica/15406 cli download report #15675

Engagement/jessica/15406 cli download report #15675

JessicaWNava commented Aug 23, 2024

github-actions bot commented Aug 23, 2024 •

edited

Loading

github-actions bot commented Aug 23, 2024 •

edited

Loading

github-actions bot commented Aug 23, 2024 •

edited

Loading

victor-chaparro Sep 4, 2024

sonarcloud bot commented Sep 6, 2024

Engagement/jessica/15406 cli download report #15675

Engagement/jessica/15406 cli download report #15675

Conversation

JessicaWNava commented Aug 23, 2024

Changes

Checklist

Testing

Process

Linked Issues

To Be Done

Specific Security-related subjects a reviewer should pay specific attention to

github-actions bot commented Aug 23, 2024 • edited Loading

Dependency Review

OpenSSF Scorecard

Scanned Manifest Files

github-actions bot commented Aug 23, 2024 • edited Loading

Test Results

github-actions bot commented Aug 23, 2024 • edited Loading

Integration Test Results

victor-chaparro Sep 4, 2024

Choose a reason for hiding this comment

sonarcloud bot commented Sep 6, 2024

Quality Gate failed

github-actions bot commented Aug 23, 2024 •

edited

Loading

github-actions bot commented Aug 23, 2024 •

edited

Loading

github-actions bot commented Aug 23, 2024 •

edited

Loading