Skip to content

Conversation

parthshah1
Copy link
Contributor

Related Issues

Adding read, write and idle timeouts in rpc handler. Fixes this bug.

Proposed Changes

Adding additional timeouts for rpc handler.

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Aug 27, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

@github-project-automation github-project-automation bot moved this from 📌 Triage to ⌨️ In Progress in FilOz Aug 27, 2025
@parthshah1
Copy link
Contributor Author

@rvagg 1 MB header limit seems way too generous imo, do we expect such large headers. Maybe 64 Kb suffice.

@rvagg rvagg changed the title fix(rpc): Adding read, write and idle timeout to rpc handler fix(rpc): add read, write and idle timeout to rpc handler Aug 28, 2025
@github-actions github-actions bot dismissed their stale review August 28, 2025 04:05

PR title now matches the required format.

@rvagg
Copy link
Member

rvagg commented Aug 28, 2025

@parthshah1 would you mind adding an entry to the changelog for this? just follow the pattern there and link back to this PR, you could even just copy and paste the PR title here (which I've modified slightly).

@BigLep
Copy link
Member

BigLep commented Aug 28, 2025

@parthshah1 parthshah1 closed this by deleting the head repository Sep 8, 2025
@BigLep
Copy link
Member

BigLep commented Sep 8, 2025

Followup happening in #13327

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

Labels

None yet

Projects

Status: ⌨️ In Progress

Development

Successfully merging this pull request may close these issues.

3 participants