enhancement: allow sse_read_timeout to be configurable

[satyamsi]

Today fast agent is using default timeout for sse_client which happens to be 5 minutes. for production systems this is too low as our observation is that with 5mins of inactivity with an mcp server it becomes unusable.

[evalstate]

Can I just check what the 5x multiplication is for here (e.g. why not just set the desired value in the read_timeout_seconds)?

[satyamsi]

Can I just check what the 5x multiplication is for here (e.g. why not just set the desired value in the read_timeout_seconds)?

There's two things:

1. read_timeout_seconds if configured is used for the session as well. You'd want the transport to have a higher number. so either we introduce a new param for this all the way into the config or use a multiplier
2. sse_client has a default time for 5*60

I have therefore just used - (configured or 60 (if not configured)) x 5 (multiplier)

If you'd like it a different way or just use configured or 5*60 I can do that as well

LMK how you'd like it.

[evalstate]

Let's add an sse_timeout to the server config and use an unadjusted value if set and a sensible default of your choosing 👍

…

On Thu, 10 Apr 2025, 23:56 Satyam Sinha, ***@***.***> wrote: Can I just check what the 5x multiplication is for here (e.g. why not just set the desired value in the read_timeout_seconds)? There's two things: 1. read_timeout_seconds if configured is used for the session as well. You'd want the transport to have a higher number. so either we introduce a new param for this all the way into the config or use a multiplier 2. sse_client has a default time for 5*60 I have therefore just used - (configured or 60 (if not configured)) x 5 (multiplier) If you'd like it a different way or just use configured or 5*60 I can do that as well LMK how you'd like it. — Reply to this email directly, view it on GitHub <#57 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOYXFRLPDGEOECAJAUDEYT2Y3ZI5AVCNFSM6AAAAAB24ZSSHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOOJVGM2DQMJTG4> . You are receiving this because you commented.Message ID: ***@***.***> *satyamsi* left a comment (evalstate/fast-agent#57) <#57 (comment)> Can I just check what the 5x multiplication is for here (e.g. why not just set the desired value in the read_timeout_seconds)? There's two things: 1. read_timeout_seconds if configured is used for the session as well. You'd want the transport to have a higher number. so either we introduce a new param for this all the way into the config or use a multiplier 2. sse_client has a default time for 5*60 I have therefore just used - (configured or 60 (if not configured)) x 5 (multiplier) If you'd like it a different way or just use configured or 5*60 I can do that as well LMK how you'd like it. — Reply to this email directly, view it on GitHub <#57 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOYXFRLPDGEOECAJAUDEYT2Y3ZI5AVCNFSM6AAAAAB24ZSSHKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOOJVGM2DQMJTG4> . You are receiving this because you commented.Message ID: ***@***.***>

[satyamsi]

Testing in progress

[evalstate]

Great addition, thanks - added to the docs here: https://fast-agent.ai/mcp/#adding-an-sse-server