Skip to content

fix: improve API call connection handling to prevent downtime #531

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.

Sign up for GitHub

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

Jump to bottom
Merged
rajnandan1 merged 2 commits into from
Jan 9, 2026
Merged

fix: improve API call connection handling to prevent downtime #531

rajnandan1 merged 2 commits into from
Jan 9, 2026
+22 −5

Conversation

@Aj7Ay
Copy link
Contributor

@Aj7Ay Aj7Ay commented Jan 5, 2026

This PR improves the API call service to prevent false downtime reports by enhancing connection handling and timeout detection.

Changes

  • ✅ Add HTTPS agent with connection pooling (keepAlive, maxSockets, maxFreeSockets)
  • ✅ Improve timeout detection to catch ECONNABORTED errors and timeout messages
  • ✅ Add better axios configuration (maxRedirects, validateStatus, maxContentLength)
  • ✅ Enhance error handling for more reliable monitoring

Benefits

  1. Connection Reuse: Reduces connection overhead by reusing existing connections
  2. Better Timeout Detection: More accurate timeout detection triggers retry mechanism
  3. Connection Pooling: Maintains idle connections for improved resilience during traffic spikes
  4. Robust Error Handling: Improved error detection ensures transient issues are properly caught and retried

Testing

  • Tested with various API endpoints
  • Verified timeout detection works correctly
  • Confirmed connection pooling improves performance

Aj7ay7 and others added 2 commits January 5, 2026 18:09
@Aj7ay7
fix: improve API call connection handling to prevent downtime
83e807a 
- Add HTTPS agent with connection pooling (keepAlive, maxSockets)
- Improve timeout detection to catch ECONNABORTED errors
- Add better axios configuration (maxRedirects, validateStatus)
- Enhance error handling for more reliable monitoring

This prevents false downtime reports by:
1. Reusing connections to reduce overhead
2. Better timeout detection triggering retry mechanism
3. Connection pooling for improved resilience during traffic spikes
@Aj7Ay
Merge pull request #1 from Aj7Ay/fix/improve-api-call-connection-hand…
c69d9fd 
…ling

fix: improve API call connection handling to prevent downtime
@Aj7Ay Aj7Ay mentioned this pull request Jan 6, 2026
Closed
@rajnandan1 rajnandan1 merged commit 5dea324 into rajnandan1:main Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rajnandan1 rajnandan1 rajnandan1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Aj7Ay @rajnandan1 @Aj7ay7