dropbox / dropbox/pygerduty

Bad URL: /incidents/{ID}/log_entries/count

Open
#80 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
165
Forks
71
PR merge metrics
No merged PRs in 30d

Description

For tracking purposes.

The [v2 REST API](https://v2.developer.pagerduty.com/docs/rest-api) no longer officially supports the `/count` path. Although empirical evidence would suggest it still exists for `/incidents/count`, there is no documentation supporting this, and plenty of [support conversations suggesting otherwise](https://community.pagerduty.com/t/statistics-and-counts-in-v2/1421/2?u=jjinno). This is _especially_ true for sub-container elements such as `Incident.log_entries`

To be v2 compliant, all `count()` calls should be changed to use [the documented `total=true` parameter](https://v2.developer.pagerduty.com/docs/pagination) on a single paginated request. The use of `limit=1` _should_ theoretically allow for the smallest possible request, though some testing on older incidents seems to suggest that this must be `limit=2` or higher to avoid HTTP errors.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.