Skip to content

fix(server): handle termination signals gracefully - #721

Merged
TS-mfon merged 4 commits into
StellarState:mainfrom
emynogithub:fix/issue-629-reconcile-settlement
Oct 30, 2026
Merged

TS-mfon merged 4 commits into
StellarState:mainfrom
emynogithub:fix/issue-629-reconcile-settlement

Conversation

@emynogithub

Copy link
Copy Markdown
Contributor

Summary

  • Handle SIGTERM and SIGINT with an idempotent shutdown path.
  • Stop accepting connections and drain requests up to HTTP_SHUTDOWN_TIMEOUT_MS, then close remaining connections.
  • Close the database connection pool after the server drains.

Closes #629

@drips-wave

drips-wave Bot commented Oct 22, 2026

Copy link
Copy Markdown

@emynogithub Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@TS-mfon

TS-mfon commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

✅ PR Ready for Review

Due to a recent malware incident affecting our repositories, all PRs are required to pass automated security checks before being reviewed for merging.

This PR has passed all automated checks:

  • ✅ Security scan — no malicious code detected
  • ✅ No merge conflicts

A maintainer will review and merge this PR shortly.

@TS-mfon

TS-mfon commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

✅ PR Ready for Review

Due to a recent malware incident affecting our repositories, all PRs are required to pass automated security checks before being reviewed for merging.

This PR has passed all automated checks:

  • ✅ Security scan — no malicious code detected
  • ✅ No merge conflicts

A maintainer will review and merge this PR shortly.

@TS-mfon

TS-mfon commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

⚠️ Merge Conflict Detected

This PR currently has a merge conflict and cannot be merged until it is resolved.

Please rebase or merge the latest changes from the base branch into your branch and push again.

@TS-mfon
TS-mfon merged commit 709bfb6 into StellarState:main Oct 30, 2026
2 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