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

Rate Limit Adjustment for RequestSignedValues #17481

Conversation

DaOneLuna
Copy link
Contributor

Adjusted the rate limit to prevent failure to non local connections.

@Starttoaster Starttoaster added the Changed Required label for PR that categorizes merge commit message as "Changed" for changelog label Feb 2, 2024
@hoffmang9
Copy link
Member

Thank you for this. Can you sign your commits?

@DaOneLuna DaOneLuna force-pushed the hb.chip22-rcblock-update branch from caa5de9 to 355197f Compare February 3, 2024 18:50
@DaOneLuna
Copy link
Contributor Author

Thank you for this. Can you sign your commits?

Should be signed now

@Starttoaster Starttoaster merged commit b8a31c1 into Chia-Network:hb.chip22-rcblock-update Feb 5, 2024
258 checks passed
Starttoaster pushed a commit that referenced this pull request Feb 14, 2024
* Send RewardChainBlockUnfinished to the harvester

- Simplify data pasing in request_signed_values

* Fix instances where the rc_block_unfinished must be sent

* Lint, fix test, generate protocol messages

* Rate Limit Adjustment for RequestSignedValues (#17481)

* Further bump request_signed_values rate limit

---------

Co-authored-by: Amine Khaldi <[email protected]>
Co-authored-by: Luna <[email protected]>
felixbrucker pushed a commit to felixbrucker/chia-blockchain that referenced this pull request Feb 16, 2024
…rk#17435)

* Send RewardChainBlockUnfinished to the harvester

- Simplify data pasing in request_signed_values

* Fix instances where the rc_block_unfinished must be sent

* Lint, fix test, generate protocol messages

* Rate Limit Adjustment for RequestSignedValues (Chia-Network#17481)

* Further bump request_signed_values rate limit

---------

Co-authored-by: Amine Khaldi <[email protected]>
Co-authored-by: Luna <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants