Skip to content

feat: configurable timeout and release script #6

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

Merged
merged 5 commits into from
Jun 24, 2025

Conversation

chriswessels
Copy link
Member

No description provided.

chriswessels and others added 5 commits June 24, 2025 14:25
- Increase default timeout from 30s to 120s for Graph Gateway requests
- Add environment variable configuration via SUBGRAPH_REQUEST_TIMEOUT_SECONDS
- Add SubgraphServer::with_timeout() method for programmatic configuration
- Include comprehensive test suite for timeout configuration scenarios
- Update documentation with timeout configuration guidance and troubleshooting

Fixes upstream request timeout errors (MCP error -32001) for complex GraphQL queries.
- Remove artificial 50ms delay from metrics handler for better performance
- Replace .unwrap() with proper error handling in metrics encoding
- Clarify in README that timeout configuration only applies to local server instances
- Update troubleshooting section to distinguish between local vs hosted service options
- Improve user experience by preventing metrics endpoint crashes on encoding errors
@sahra-karakoc sahra-karakoc force-pushed the chris/configurable-timeout branch from 574d248 to 78edae3 Compare June 24, 2025 13:22
@sahra-karakoc sahra-karakoc merged commit fe25322 into main Jun 24, 2025
3 checks passed
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