Skip to content

Enable setting ServerTimeout and KeepAliveInterval in HubConnectionBuilder#46065

Merged
surayya-MS merged 15 commits into
dotnet:mainfrom
surayya-MS:hubConnectionBuilder
Jan 19, 2023
Merged

Enable setting ServerTimeout and KeepAliveInterval in HubConnectionBuilder#46065
surayya-MS merged 15 commits into
dotnet:mainfrom
surayya-MS:hubConnectionBuilder

Conversation

@surayya-MS
Copy link
Copy Markdown
Member

@surayya-MS surayya-MS commented Jan 12, 2023

Enable setting ServerTimeout and KeepAliveInterval in HubConnectionBuilder

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Added withServerTimeout and withKeepAliveInterval in HubConnectionBuilder for ts and csharp clients

Fixes #18840 and partially #44742

@ghost ghost added the area-signalr Includes: SignalR clients and servers label Jan 12, 2023
Comment thread src/SignalR/clients/csharp/Client.Core/src/HubConnection.cs Outdated
Comment thread src/SignalR/clients/csharp/Client.Core/src/HubConnectionOptions.cs Outdated
Comment thread src/SignalR/clients/ts/signalr/src/HubConnectionBuilder.ts Outdated
Comment thread src/SignalR/clients/ts/signalr/src/HubConnectionBuilder.ts Outdated
Comment thread src/SignalR/clients/csharp/Client.Core/src/HubConnectionBuilderExtensions.cs Outdated
surayya-MS and others added 6 commits January 17, 2023 19:49
…s.cs

Co-authored-by: Brennan <brecon@microsoft.com>
Co-authored-by: Brennan <brecon@microsoft.com>
Co-authored-by: Brennan <brecon@microsoft.com>
…rExtensions.cs

Co-authored-by: Brennan <brecon@microsoft.com>
…ubConnection

2. Added test to HubConnectionBuilderTests
Comment thread src/SignalR/clients/ts/signalr/src/HubConnectionBuilder.ts Outdated
Comment thread src/SignalR/clients/ts/signalr/src/HubConnectionBuilder.ts Outdated
surayya-MS and others added 2 commits January 18, 2023 18:44
Co-authored-by: Brennan <brecon@microsoft.com>
Co-authored-by: Brennan <brecon@microsoft.com>
Copy link
Copy Markdown
Member

@halter73 halter73 left a comment

Choose a reason for hiding this comment

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

Do we want to do the same thing in HttpHubConnectionBuilder.java? If we do, it doesn't have to be in this PR.

Comment thread src/SignalR/clients/ts/signalr/src/HubConnection.ts Outdated
@surayya-MS
Copy link
Copy Markdown
Member Author

Do we want to do the same thing in HttpHubConnectionBuilder.java? If we do, it doesn't have to be in this PR.

Yes. It will be in another PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-signalr Includes: SignalR clients and servers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Customizable serverTimeoutInMilliseconds in Server-Side Blazor

3 participants