Redesign FP Date Shift Architecture#1497
Open
trobanga wants to merge 3 commits intoepic/fhir-pseudonymizerfrom
Open
Redesign FP Date Shift Architecture#1497trobanga wants to merge 3 commits intoepic/fhir-pseudonymizerfrom
trobanga wants to merge 3 commits intoepic/fhir-pseudonymizerfrom
Conversation
trobanga
force-pushed
the
epic/fhir-pseudonymizer
branch
from
2502019 to
392d3a4
Compare
trobanga
force-pushed
the
epic/fhir-pseudonymizer
branch
from
392d3a4 to
1139e1e
Compare
trobanga
force-pushed
the
fp-date-shift
branch
2 times, most recently
from
e7e8ef2 to
650ba78
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## epic/fhir-pseudonymizer #1497 +/- ##
==========================================================
Coverage ? 99.18%
Complexity ? 691
==========================================================
Files ? 140
Lines ? 2450
Branches ? 86
==========================================================
Hits ? 2430
Misses ? 2
Partials ? 18
🚀 New features to boost your workflow:
|
Add GpasProxyController that mimics gPAS's $pseudonymizeAllowCreate but returns transport IDs instead of real pseudonyms. This allows the external FHIR Pseudonymizer to call TCA (not gPAS directly), maintaining data isolation via transport IDs. Also add E2E test infrastructure for the FHIR Pseudonymizer flow: - FP compose config pointing at TCA proxy with TLS and basic auth - Anonymization config, project config, and expected results - CI workflow steps for the fp-example transfer test Fix server cert generation to include Subject Alternative Names (SANs), required by .NET's strict TLS hostname validation.
Add agent-level E2E tests for the FHIR Pseudonymizer flow: - GpasProxyE2E: verifies gPAS proxy returns transport IDs not real pseudonyms - FpTransportMappingE2E: tests two-phase proxy + consolidation flow - FhirPseudonymizerE2E: full CDA transfer with FP WireMock, consent, and transport mapping consolidation
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
DateShiftAnonymizerutilityDateShiftControllerwithFpTransportMappingControllerthat consolidates identity tIDs and date mappings in a single callTransportIdServiceby removing date-shift-specific logicFhirPseudonymizerStepwith external FHIR Pseudonymizer integration