Skip to content

README seems innaccurate - doesTxNeedFrontrunningProtection doesn't check gas > 42,000 #138

@samlaf

Description

@samlaf

The README, in this section, says

  • Does the transaction use more than 42,000 gas? If it doesn't then the Flashbots Relay will reject it, and we're not aware of use cases that use such low gas that need frontrunning protection. Thus, we send low gas transactions to the mempool.

However, doesTxNeedFrontrunningProtection only logs the gas usage but doesn't check that the gasLimit is > 42,000. I also ctrl-f'ed the repo for 42000, 42,000, 42_000` and couldn't find anything.

Is the readme bullet point still accurate?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions