-
Notifications
You must be signed in to change notification settings - Fork 38
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
PATCH Retries ihev2 #2291
PATCH Retries ihev2 #2291
Conversation
@@ -137,13 +137,21 @@ export async function sendSignedXmlMtom({ | |||
}, | |||
httpsAgent: agent, | |||
responseType: "arraybuffer", | |||
maxBodyLength: Infinity, | |||
maxContentLength: Infinity, |
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.
maxContentLength size of -1 exceeded
^ we have had this error in prod 3 times
We added retries for ERR_BAD_RESPONSE, and this change was also recommended in this issue
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.
was missing these from previous PR
@@ -111,6 +112,7 @@ export class IHEGatewayV2Async extends IHEGatewayV2 { | |||
for (const chunk of requestChunks) { | |||
const params = { patientId, cxId, requestId, drRequestsGatewayV2: chunk }; | |||
|
|||
await sleep(1000); |
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.
delay between DR request attempts. This is within a single gateway.
Refs: #1667 Signed-off-by: Jonah Kaye <[email protected]>
e45834d
to
2ac2825
Compare
packages/core/src/external/carequality/ihe-gateway-v2/ihe-gateway-v2-async.ts
Outdated
Show resolved
Hide resolved
packages/core/src/external/carequality/ihe-gateway-v2/saml/saml-client.ts
Outdated
Show resolved
Hide resolved
Refs: #1667 Signed-off-by: Jonah Kaye <[email protected]>
packages/core/src/external/carequality/ihe-gateway-v2/ihe-gateway-v2-async.ts
Outdated
Show resolved
Hide resolved
packages/core/src/external/carequality/ihe-gateway-v2/ihe-gateway-v2-async.ts
Outdated
Show resolved
Hide resolved
packages/core/src/external/carequality/ihe-gateway-v2/saml/saml-client.ts
Outdated
Show resolved
Hide resolved
Refs: #1667 Signed-off-by: Jonah Kaye <[email protected]>
Refs: #1667 Signed-off-by: Jonah Kaye <[email protected]>
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.
const SLEEP_IN_BETWEEN_DOCUMENT_RETRIEVAL_REQUESTS = dayjs | ||
.duration({ seconds: 1 }) | ||
.asMilliseconds(); |
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.
Its better to create the const as duration and convert to the unit needed on the place where we use it
Refs: #1667 Signed-off-by: Jonah Kaye <[email protected]>
Ticket: #1667
Description
Testing
Release Plan
master