celestiaorg / orchestrator-relayer

Go implementation of the Blobstream orchestrator and relayer
Apache License 2.0
29 stars 17 forks source link

feat: support eip1559 when speeding up transactions #644

Closed rach-id closed 11 months ago

rach-id commented 11 months ago

Overview

Previously, we only supported speeding up legacy transactions. With this, we can also speed up EIP1559 ones.

Checklist

Summary by CodeRabbit

coderabbitai[bot] commented 11 months ago

Walkthrough

The codebase has been updated to better handle Ethereum transactions, specifically by introducing new functions to manage the resubmission of transactions with increased gas fees. This includes separate handling for EIP-1559 dynamic fee transactions and legacy transactions, ensuring compatibility with the Ethereum network's evolving gas pricing mechanisms.

Changes

File(s) Change Summary
relayer/relayer.go Introduced functions for creating speeded-up transactions for both EIP-1559 and legacy types, and updated waitForTransactionAndRetryIfNeeded to handle resubmission with adjusted gas fees.

🐇✨ In the code's burrow, deep and wide, New functions spring to life inside. With swifter gas, transactions fly, As CodeRabbit's work reaches the sky. 🚀🌌 🐇✨


Tips ### Chat with CodeRabbit Bot (`@coderabbitai`) - If you reply to a *review comment* from CodeRabbit, the bot will automatically respond. - To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment - Note: Review comments are made on code diffs or files, not on the PR overview. - Add `@coderabbitai ignore` anywhere in the PR description to prevent this PR from being reviewed. ### CodeRabbit Commands (invoked as PR comments) - `@coderabbitai pause` to pause the reviews on a PR. - `@coderabbitai resume` to resume the paused reviews. - `@coderabbitai review` to trigger a review. This is useful when automatic reviews are disabled for the repository. - `@coderabbitai resolve` resolve all the CodeRabbit review comments. - `@coderabbitai help` to get help. Note: For conversation with the bot, please use the review comments on code diffs or files. ### CodeRabbit Configration File (`.coderabbit.yaml`) - You can programmatically configure CodeRabbit by adding a `.coderabbit.yaml` file to the root of your repository. - The JSON schema for the configuration file is available [here](https://coderabbit.ai/integrations/coderabbit-overrides.v2.json). - If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: `# yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json`
codecov-commenter commented 11 months ago

Codecov Report

Attention: 86 lines in your changes are missing coverage. Please review.

Comparison is base (2aa4b32) 0.00% compared to head (2adb86d) 24.64%.

Files Patch % Lines
relayer/relayer.go 0.00% 86 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #644 +/- ## ========================================= + Coverage 0 24.64% +24.64% ========================================= Files 0 29 +29 Lines 0 3201 +3201 ========================================= + Hits 0 789 +789 - Misses 0 2317 +2317 - Partials 0 95 +95 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.