CDCgov / prime-reportstream

ReportStream is a public intermediary tool for delivery of data between different parts of the healthcare ecosystem.
https://reportstream.cdc.gov
Creative Commons Zero v1.0 Universal
69 stars 39 forks source link

HHSProtect - Settings Changes needed for adding new Senders #4255

Open rhood23699 opened 2 years ago

rhood23699 commented 2 years ago

Currently, every new Sender added to ReportStream must send its production data to the "hhsprotect" sender. A bit more explanation:

  1. ReportStream is to send all production data to HHS protect EXCEPT for those senders that reject this.
  2. Simple Report is one of the senders that CANNOT send its data to HHS protect.
  3. We will handle any rejections by adding the specific sender to the jurisdiction filter for filtering.
  4. Currently, the setup was to add each sender to the jurisdiction filter, which will result in a big long list and extra effort as we onboard more senders. THe need here is to simplify this setup, so we do not have to update this filter over and over.
  5. IMPORTANT: We must exclude any senders not currently sending data to HHS protect, so it cannot be a blanket everyone sends their data there.

Things know

Example from hhsprotect settings: jurisdictionalFilter: - "matches(sender_id, SafeHealth,ImageMover,Cue,InBios,Strac,.CueHlth.,reddyfmc,primary,cuc-al,guc-la,abbott,lifepoint,hca,pmg,ihealthlabs,mns)" qualityFilter:

Acceptance criteria

  1. All results with processing_mode_code P (production) are sent to HHS protect.
  2. Results from senders CURRENTLY not sending data to HHS protect are NOT sent to HHS protect. This means we must reverse the current list of senders from a match to does not match.
carlosfelix2 commented 2 years ago

I went into PROD and did some magic to generate the list of senders that will be EXCLUDED. Can you all please look at this and make sure it is right?

"hollywoodsmiles-ca"
"mns"
"kern-county-phsd-ca"
"cdphe-csvuploader"
"cadph-csvuploadertest"
"sourcebioscience_ca"
"warrior-dx-co"
"covidclinic-org"
"naval-health-clinic-lemoore"
"all-in-one-health-ca"
"vivature-tx"
"sdilabs-ca"
"johns_hopkins"
"ocrapidtest-ca"
"waters"
"simple_report"
"stemexpress-ca"
"corebiolabs-oh"
"pmg"

Here is the current match string in PROD: SafeHealth,ImageMover,Cue,InBios,Strac,.*CueHlth.*,reddyfmc,primary,cuc-al,guc-la,abbott,lifepoint,hca

rhood23699 commented 2 years ago

sender_id: "all-in-one-health-ca" - They are in production and should be included. I'll do that today.

Below are Senders that I'm working on that will soon need to be included. Note, some of the Senders in your list are not mine, so I can't really answer for them. "hollywoodsmiles-ca" "kern-county-phsd-ca" "cdphe-csvuploader" "cadph-csvuploadertest" "sourcebioscience_ca" "warrior-dx-co" "covidclinic-org" "naval-health-clinic-lemoore" "vivature-tx" "sdilabs-ca" "ocrapidtest-ca" "stemexpress-ca" "corebiolabs-oh"

JGreenFearless commented 2 years ago

Hey team! Please add your planning poker estimate with ZenHub @carlosfelix2 @jh765 @jimduff-usds @oslynn @smythe5 @whytheplatypus

JGreenFearless commented 2 years ago

@rhood23699 Is this something that still needs to be addressed?

JGreenFearless commented 2 years ago

@JGreenFearless to follow-up with O&O to see if this is still an issue that needs to be resolved.