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

REST API limited to 30 responses #25

Open
jacob-keller opened this issue Oct 25, 2023 · 1 comment
Open

REST API limited to 30 responses #25

jacob-keller opened this issue Oct 25, 2023 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@jacob-keller
Copy link

The current implementation of the REST API doesn't handle pagination responses for listing patches, so it is limited to a single reply, which is 30 items by default on most patchwork servers.

@jacob-keller
Copy link
Author

Supporting pagination looks like it requires parsing the Link header response and properly iterating through the pages. I don't think urllib supports handling Link headers, but the requests library could be used instead of urllib and that includes built-in support for at least parsing Link headers which would make this easier to implement.

@stephenfin stephenfin added enhancement New feature or request good first issue Good for newcomers labels Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants