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

rpcserver: Limit getstakeversioninfo count. #2221

Merged

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Jun 3, 2020

This is rebased on #2218.

This modifies the RPC server handler for getstakeversioninfo to limit any provided interval counts to the maximum number of intervals that are actually available for retrieval and improves a couple of error messages while here.

Resolves #2220.

@davecgh davecgh added this to the 1.6.0 milestone Jun 3, 2020
Copy link
Member

@degeri degeri left a comment

Choose a reason for hiding this comment

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

LGTM . This resolves #2220

This modifies the RPC server handler for getstakeversioninfo to limit
any provided interval counts to the maximum number of intervals that are
actually available for retrieval and improves a couple of error
messages while here.
@davecgh davecgh force-pushed the rpcserver_limit_getstakeversioninfo_count branch from 26a95cb to 3f43437 Compare June 4, 2020 21:13
@davecgh davecgh merged commit 3f43437 into decred:master Jun 4, 2020
@davecgh davecgh deleted the rpcserver_limit_getstakeversioninfo_count branch June 4, 2020 22:44
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.

[RPC] out of memory in getstakeversioninfo
4 participants