Ticket 72 handle IMA 2 IMA net errors #83
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IMA
uses express-based HTTP server to deliver IMA-2-IMA notifications like PWA state tracking. This PR fixes network connection problems by adding error handlers. Express-related code also ported from JS into TypeScript. No performance changes expected.It's barely possible to cover all possible network problems with unit-tests, especially when connection problem appear after JSON request received and before answer is sent. Such errors can have guaranteed emulation only with modified source code of IMA Agent where network socket is manually damaged.
New version's testing sequence and expected result on
dev net
is:20-30
minutes163
Old buggy version's notes: