Closed arran-standish closed 1 year ago
Patch coverage: 91.66%
and project coverage change: +0.02%
:tada:
Comparison is base (
4ab86ea
) 86.81% compared to head (68ae11d
) 86.83%.:exclamation: Current head 68ae11d differs from pull request most recent head cade24e. Consider uploading reports for the commit cade24e to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
If OpenHIM crashes with a request in-flight (so before the down stream service responds) this transaction will stay in 'Processing' forever and does not provide a true reflection of the transaction state.
So on startup it will find all transactions in that state and update them to failed. It uses mongoose's cursor functionality which returns a stream and so shouldn't interfere with the main application as it will only process once the stream receives data. It also properly respects node's event loop and so won't block it (i.e.: incoming requests still get processed normally).