Skip to content

Gasless transaction fixes#45

Merged
codchen merged 1 commit intomasterfrom
tony-chen-gasless
Jun 16, 2022
Merged

Gasless transaction fixes#45
codchen merged 1 commit intomasterfrom
tony-chen-gasless

Conversation

@codchen
Copy link
Collaborator

@codchen codchen commented Jun 15, 2022

This PR does two things:

  1. Add a whitelist (currently hardcoded but will be parameterized in a subsequent PR) for addresses that can cancel orders gaslessly.
  2. Ensure order placements and order cancellations (for whitelisted accounts) are truly gasless by overriding the gas meter in TX's context to an infinite gas meter

Tested on local chain

@codchen codchen requested review from LCyson and philipsu522 June 15, 2022 23:31
@codchen codchen force-pushed the tony-chen-gasless branch from 9438a5a to 545133f Compare June 15, 2022 23:31
Copy link
Contributor

@philipsu522 philipsu522 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice nice

@codchen codchen merged commit dc74ace into master Jun 16, 2022
masih pushed a commit that referenced this pull request Sep 26, 2025
masih pushed a commit that referenced this pull request Oct 1, 2025
* Keep LRU cache of recent peers and retry if we run out

* Add go
masih pushed a commit that referenced this pull request Oct 1, 2025
masih pushed a commit that referenced this pull request Oct 1, 2025
masih pushed a commit that referenced this pull request Oct 9, 2025
@masih masih deleted the tony-chen-gasless branch October 31, 2025 15:53
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