Skip to content

Http server timeouts - #814

Merged
hkalodner merged 2 commits into
masterfrom
http-server-timeouts
Jul 20, 2022
Merged

Http server timeouts#814
hkalodner merged 2 commits into
masterfrom
http-server-timeouts

Conversation

@Tristan-Wilson

Copy link
Copy Markdown
Member

Make externally facing srv timeouts configurable

Make timeouts on Nitro RPC, and DAS RPC/REST servers configurable.

Also remote defunct dasrest executable.

Make timeouts on Nitro RPC, and DAS RPC/REST servers configurable.

Also remote defunct dasrest executable.
Resolve conflicts around manually set timeouts on DAS RPC/REST servers

@hkalodner hkalodner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hkalodner
hkalodner enabled auto-merge (rebase) July 20, 2022 14:40
@codecov

codecov Bot commented Jul 20, 2022

Copy link
Copy Markdown

Codecov Report

Merging #814 (e2829cc) into master (020c32a) will decrease coverage by 0.05%.
The diff coverage is 55.17%.

@@            Coverage Diff             @@
##           master     #814      +/-   ##
==========================================
- Coverage   52.91%   52.85%   -0.06%     
==========================================
  Files         223      222       -1     
  Lines       27161    27130      -31     
  Branches      494      494              
==========================================
- Hits        14372    14340      -32     
+ Misses      11250    11246       -4     
- Partials     1539     1544       +5     

auto-merge was automatically disabled July 20, 2022 14:58

Rebase failed

@hkalodner
hkalodner merged commit efd41aa into master Jul 20, 2022
@hkalodner
hkalodner deleted the http-server-timeouts branch July 20, 2022 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants