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.

Sign up for GitHub

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

Jump to bottom

Reputation #43

Merged
merged 20 commits into from
Feb 1, 2023
Merged

Reputation #43

merged 20 commits into from
Feb 1, 2023

Conversation

Vid201
Copy link
Member

@Vid201 Vid201 commented Jan 22, 2023 •
edited
Loading

Several sanity checks and simulation conditions are based on the reputation of entities (paymasters, senders, aggregators, factories). Thus we should have a reputation implementation first.

This PR contains:

  • reputation management
  • debug endpoints
  • small changes to the Makefile

@Vid201 Vid201 linked an issue Jan 22, 2023 that may be closed by this pull request
Implement reputation scoring and throttling/banning for global entities #30
Closed
@Vid201
feat: add reputation primitives
397360b
@Vid201
chore: make reputation generic
acc86bc
@Vid201
feat: add update hourly function to reputation
ace7119
@Vid201
feat: implement several methods related to reputation
21255ac
@Vid201
feat: add get status method for reputation
e6552e3
@Vid201
chore: fix lint
54dc188
@Vid201
feat: add verify stake method for reputation
2085c2a
@Vid201
chore: fix lint
d8e3297
@Vid201
chore: call repeat hourly
a5ebd12
@Vid201
feat: debug methods for reputation
4aab56f
@Vid201
chore: move two mempool methods to debug only
3f0ec57
@Vid201
feat: add debug api
c9b6a77
@Vid201
feat: make reputation in uopoolservice generic, implement clear endpoint
a372772
@Vid201
feat: implement debug get all user operations
f46d4f1
@Vid201
feat: implement debug reputation endpoints
c2ea432
@Vid201
feat: reputation different for each entry point
539c459
@Vid201
feat: add reputation tests
46733fa
@Vid201
chore: update Makefile
80fd7e5
@Vid201 Vid201 marked this pull request as ready for review January 30, 2023 11:37
@Vid201 Vid201 changed the title Draft: reputation Reputation Jan 30, 2023
@Vid201 Vid201 requested a review from zsluedem January 30, 2023 11:37
zsluedem
zsluedem reviewed Jan 31, 2023
View reviewed changes
Copy link
Collaborator

@zsluedem zsluedem left a comment

Choose a reason for hiding this comment

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

Good work! I left some comments and questions.

bin/bundler-rpc.rs Show resolved Hide resolved
bin/bundler.rs Show resolved Hide resolved
src/uopool/mod.rs Show resolved Hide resolved
src/uopool/mod.rs Show resolved Hide resolved
zsluedem
zsluedem approved these changes Feb 1, 2023
View reviewed changes
Copy link
Collaborator

@zsluedem zsluedem left a comment

Choose a reason for hiding this comment

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

Nice work.

@Vid201 Vid201 merged commit 0ad243a into main Feb 1, 2023
@Vid201 Vid201 deleted the feat/reputation branch February 1, 2023 09:19
@zsluedem zsluedem mentioned this pull request Mar 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zsluedem zsluedem zsluedem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement reputation scoring and throttling/banning for global entities
2 participants
@Vid201 @zsluedem