apiFetch should be able to handle a 200 response code with an empty body
#21 899 ouverte le 26 avr. 2020
Métriques du dépôt
- Stars
- (9 607 stars)
- Métriques de merge PR
- (Merge moyen 12j 18h) (509 PRs mergées en 30 j)
Description
Describe the bug
If the response is 200 but the body is empty, it causes an error:
The code handles 204 responses, but not 200 with empty body: https://github.com/WordPress/gutenberg/blob/%40wordpress/api-fetch%403.13.1/packages/api-fetch/src/utils/response.js#L16-L20
From https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/204
The common use case is to return 204 as a result of a PUT request, updating a resource, without changing the current content of the page displayed to the user. If the resource is created, 201 Created is returned instead. If the page should be changed to the newly updated page, the 200 should be used instead.
So, since a 200 with empty body isn't a wrong response, the apiFetch should handle it instead of throwing a try/catch error.
https://mcculloughwebservices.com/2016/09/23/handling-a-null-response-from-an-api/ might be helpful.