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

limit max transactions in NotificationMessage and calculate known_transactions LRU_CACHE_SIZE size #2067

Open
jolestar opened this issue Jan 26, 2021 · 0 comments · Fixed by #2117
Open

limit max transactions in NotificationMessage and calculate known_transactions LRU_CACHE_SIZE size #2067

jolestar opened this issue Jan 26, 2021 · 0 comments · Fixed by #2117
Assignees
@sanlee42
Labels
security

Comments

@jolestar
Copy link
Member

  1. limit max transactions in NotificationMessage::Transactions.
  2. known_transactions LRU_CACHE_SIZE > the transactions size
@jolestar jolestar added this to the starcoin-1.0-beta milestone Jan 26, 2021
@jolestar jolestar linked a pull request Feb 4, 2021 that will close this issue
[network] set network known hash lru cache to 10240 and deny integer arithmetic in network crate. #2117
Merged
@jolestar jolestar self-assigned this Feb 4, 2021
@jolestar jolestar assigned sanlee42 and unassigned jolestar Aug 23, 2021
@jolestar jolestar removed this from the starcoin-1.0 milestone Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sanlee42 sanlee42

Labels
security
Projects
None yet
Milestone
No milestone
2 participants
@jolestar @sanlee42