Bad URL: /incidents/{ID}/log_entries/count
- 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.