Skip to content

feat(keepalive): expose tcp keep-alive options#1583

Merged
jsdw merged 10 commits intoparitytech:masterfrom
nuel77:master
Aug 11, 2025
Merged

feat(keepalive): expose tcp keep-alive options#1583
jsdw merged 10 commits intoparitytech:masterfrom
nuel77:master

Conversation

@nuel77
Copy link
Contributor

@nuel77 nuel77 commented Jun 17, 2025

Exposes TCP keepalive options to the base client.

related to this issue.
opening a pr to expose the following options upstream from the internal http-connector.

  • tcp_keepalive_interval
  • tcp_keepalive_duration
  • tcp_keepalive_retires

@nuel77 nuel77 requested a review from a team as a code owner June 17, 2025 13:38
@nuel77 nuel77 changed the title expose tcp keep-alive options feat(keepalive): expose tcp keep-alive options Jun 17, 2025
@nuel77
Copy link
Contributor Author

nuel77 commented Jul 3, 2025

Hi @jsdw, can you let me know if you need any changes to this ?

@nuel77 nuel77 requested a review from jsdw July 4, 2025 12:40
@jsdw
Copy link
Collaborator

jsdw commented Jul 4, 2025

Hi @jsdw, can you let me know if you need any changes to this ?

Sorry, have been busy with a bunch of other things atm!

I would personally want to add it on the ws client/async client as well to be consistent in the repo

I would do this in this PR rather than a separate one because I'd rather that things were in a consistent state, and it would be easy to forget the follow-up and leave things in a slightly weird place :)

@jsdw
Copy link
Collaborator

jsdw commented Aug 11, 2025

I had a look and doesn't look like it makes sense currently to add those options elsewhere, so will get this merged as is!

@jsdw jsdw merged commit c0b947f into paritytech:master Aug 11, 2025
11 checks passed
@jsdw jsdw mentioned this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants