-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BCDA-8230: Remove hashed MBI #982
base: main
Are you sure you want to change the base?
Conversation
@@ -1,125 +1,98 @@ | |||
$ANSIBLE_VAULT;1.1;AES256 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed BB_HASH_ITER and BB_HASH_PEPPER
@@ -66,7 +64,7 @@ type APIClient interface { | |||
GetExplanationOfBenefit(jobData models.JobEnqueueArgs, patientID string, claimsWindow ClaimsWindow) (*fhirModels.Bundle, error) | |||
GetPatient(jobData models.JobEnqueueArgs, patientID string) (*fhirModels.Bundle, error) | |||
GetCoverage(jobData models.JobEnqueueArgs, beneficiaryID string) (*fhirModels.Bundle, error) | |||
GetPatientByIdentifierHash(jobData models.JobEnqueueArgs, hashedIdentifier string) (string, error) | |||
GetPatientByMbi(jobData models.JobEnqueueArgs, mbi string) (string, error) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any argument to keep GetPatientByIdentifierHash
functionality? Or is this a known permanent change?
params.Set("excludeSAMHSA", "true") | ||
|
||
updateParamWithServiceDate(¶ms, claimsWindow) | ||
updateParamWithLastUpdated(¶ms, jobData.Since, jobData.TransactionTime) | ||
|
||
u, err := bbc.getURL("Claim", params) | ||
u, err := bbc.getURL("Claim/_search", params) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use getURL above as well. Some nitpicks/thoughts:
- We should return the same thing each time (we return empty string above and nil here)
- Do we want to return anything more useful than an empty string? Looking at getURL it is already catching an error, maybe getURL doesnt need to return an error?
🎫 Ticket
https://jira.cms.gov/browse/BCDA-8230
🛠 Changes
ℹ️ Context
🧪 Validation