Skip to content
This repository has been archived by the owner on Jan 27, 2022. It is now read-only.

Handle pagination in response from pagerduty #1

Open
sciurus opened this issue Jul 26, 2018 · 0 comments
Open

Handle pagination in response from pagerduty #1

sciurus opened this issue Jul 26, 2018 · 0 comments

Comments

@sciurus
Copy link
Contributor

sciurus commented Jul 26, 2018

If our query for acknowledged pagerduty incidents returns more than 100 results, we'll only look at the first 100. This mean statuspage and pagerduty would get out of sync. We should handle this by refetching results with higher offsets until there are no more, then returning them all.

https://v2.developer.pagerduty.com/docs/pagination

https://github.com/mozilla-services/pagerstatus/blob/master/chalicelib/pagerduty.py#L15-L27

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant