Skip to content
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

fix: fallback to resending every 120s latest if current RPC block is stuck #809

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Dec 11, 2024

PR-Codex overview

This PR updates the logic for determining when to resend a transaction in the mine-transaction-worker.ts. It modifies how elapsed time and blocks are calculated and introduces a new condition for resending transactions based on time and block thresholds.

Detailed summary

  • Renamed ellapsedMs to elapsedSeconds and changed its calculation.
  • Updated log message to reflect the new elapsed time in seconds.
  • Changed the condition for resending a transaction to include a check for 120 seconds or minEllapsedBlocksBeforeRetry.
  • Renamed ellapsedBlocks to elapsedBlocks for clarity.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@arcoraven arcoraven merged commit b2a0b1a into main Dec 11, 2024
5 of 6 checks passed
@arcoraven arcoraven deleted the ph/retryIfEllapsedTime branch December 11, 2024 02:51
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