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

Pending transaction limit #850

Merged
merged 6 commits into from
Jan 17, 2025
Merged

Pending transaction limit #850

merged 6 commits into from
Jan 17, 2025

Conversation

mvandeberg
Copy link
Member

@mvandeberg mvandeberg commented Jan 15, 2025

Resolves #848

Brief description

Adds and enforces a per address pending transaction limit.

Integration tests will not pass without koinos/koinos-integration-tests#80 and koinos/koinos-mempool#112. I have built and pushed images for these PRs for local testing of the integration tests.

Checklist

  • I have built this pull request locally
  • I have ran the unit tests locally
  • I have manually tested this pull request
  • I have reviewed my pull request
  • I have added any relevant tests

Demonstration

@mvandeberg mvandeberg requested a review from sgerbino January 15, 2025 22:41
@mvandeberg mvandeberg marked this pull request as ready for review January 16, 2025 21:18
@sgerbino sgerbino merged commit 72af9c8 into master Jan 17, 2025
1 of 2 checks passed
@sgerbino sgerbino deleted the 848-pending-trx-limit branch January 17, 2025 18:14
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.

[FEATURE]: Check pending transaction count
2 participants