-
Notifications
You must be signed in to change notification settings - Fork 8
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
Pagination of Porter responses #3
Comments
@derekpierre is this done in nucypher/nucypher@f384c7e? |
This issue is still outstanding. The referenced commit is related to pagination of the list of stakers obtained from the smart contract i.e. the back end call. However, this issue is specific to the front end side (endpoint response to the request) i.e. if the user's request asks for something like 500 ursulas, do we return all 500 in a single large response result (as we do now), or do we limit the number that can be returned in a single response and allow for follow-on requests to obtain the remainder of ursulas. One of the concerns here is having to cache sampling results for a period of time as opposed to being totally stateless - see nucypher/nucypher#2666 (comment). |
To clarify, this didn't seem absolutely necessary for an MVP. Do you feel differently? |
@derekpierre is this done? |
No. This issue is around the question of what if someone wants to sample for a large quantity of nodes - do we return all the information in one response (which is what we do now), or split the response into paginated responses. For example, if someone called |
Related to nucypher/nucypher#2703.
If the responses for Alice endpoints
/get_ursulas
and potentially/revoke
can be too large, we can think about paginating the response. See nucypher/nucypher#2666 (comment) for some context.The text was updated successfully, but these errors were encountered: