Issue #6167 - Review failure cases of HttpOutput.Interceptor

[lachlan-roberts]

Issue #6167

This PR adds extra testing around what happens if an HttpOutput.Interceptor throws or fails the callback.

Some changes were made to HttpOutput so that in most cases we will respond with a 500 response instead of just aborting the connection.

[gregw]

Considering @lachlan-roberts research regarding this, I am switching this PR back to draft and Lachlan should work on a version that returns false from isReady if there is an exception that will be passed to onError.

@lachlan-roberts can you write up your findings either here on in the actual issue.

[stale[bot]]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions[bot]]

This pull request has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions[bot]]

This pull request has been closed due to it having no activity.