OctopusDeploy / Halibut

| Public | A secure communication stack for .NET using JSON-RPC over SSL.
Other
12 stars 44 forks source link

Adding tests with RelyOnConnectionTimeoutsInsteadOfPollingRequestMaximumMessageProcessingTimeout turned on #579

Closed sburmanoctopus closed 9 months ago

sburmanoctopus commented 9 months ago

[sc-59506]

Background

We wanted some tests to prove that RelyOnConnectionTimeoutsInsteadOfPollingRequestMaximumMessageProcessingTimeout works in general.

We cannot just turn it on by default yet, as we want the 'off' path to be tested thoroughly. Mainly because primarily, we are still using in production with this setting off.

How to review this PR

Quality :heavy_check_mark:

Pre-requisites

shortcut-integration[bot] commented 9 months ago

This pull request has been linked to Shortcut Story #59506: Decouple the Polling Response Timeout from the duration it takes to transfer a request and response.