Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Timeout Settings for gRPC and Streaming Servers #205

Open
so-sahu opened this issue Feb 26, 2024 · 0 comments · May be fixed by #221
Open

Implement Timeout Settings for gRPC and Streaming Servers #205

so-sahu opened this issue Feb 26, 2024 · 0 comments · May be fixed by #221
Assignees
Labels
enhancement New feature or request prio-mid

Comments

@so-sahu
Copy link
Contributor

so-sahu commented Feb 26, 2024

Description

The current implementations of our gRPC and streaming servers do not define timeout settings, which means that there is no timeout for these servers. This lack of timeouts can make our application vulnerable to long-lived connections and potential resource exhaustion. To address this issue, we need to implement timeout settings for both the gRPC and streaming servers.

Proposed Solution

  1. Define appropriate timeout settings for both gRPC and streaming servers.
  2. Implement these timeout settings in the server configurations.
  3. Ensure that timeouts are set at reasonable intervals to protect the application from long-lived connections.

Motivation

This change will help improve the overall security and stability of our application by preventing potential resource exhaustion due to long-lived connections.

Ref: #196 (comment)

@so-sahu so-sahu added enhancement New feature or request prio-mid labels Feb 26, 2024
@so-sahu so-sahu self-assigned this Feb 26, 2024
@github-project-automation github-project-automation bot moved this to Todo in Compute Feb 26, 2024
@so-sahu so-sahu linked a pull request Mar 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request prio-mid
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

1 participant