Conversation
|
This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket: VED-240 |
…ital/immunisation-fhir-api into VED-240-handle-rate-limit
…ital/immunisation-fhir-api into VED-240-handle-rate-limit
…ital/immunisation-fhir-api into VED-240-handle-rate-limit
…ital/immunisation-fhir-api into VED-240-handle-rate-limit
…ital/immunisation-fhir-api into VED-240-handle-rate-limit
|
dlzhry2nhs
left a comment
There was a problem hiding this comment.
Nice work. Thanks for all the changes.
Also please mention to @mfjarvis in standup that you moved some files around. Believe he was looking at improving the authentication part for the performance tests. |
To be clear, this does not mean block the PR. You have had 2 reviews so can merge if you like. I was just saying that we should give Matt a heads up about this change; which we did in standup. |



Summary
Implemented improved HTTP error handling and retry logic for PDS client integration to support clearer failure diagnostics and resilience against transient downstream failures.
Also made the mns error handling function reusable and moved to shared, so it can be re-used for PDS
Reviews Required
Review Checklist
ℹ️ This section is to be filled in by the reviewer.