SyntaxError at JSON.parse due to status=204 (no content)

[ne-spal]

Because of

else if (!body) {
          this._bodyText = ''

and

this.text().then(JSON.parse)

[mislav]

I think this is expected. If you try to parse content with JSON, but the response body isn't valid JSON (such as an empty body) then the exception should be thrown.

[ne-spal]

Yeah, but is it better than just getting "null"?

[mislav]

Yes it's very much better. Because when you're expecting JSON, you won't be expecting null. So it's better to have a descriptive exception that you can handle in exception-handling code, instead of trying to iterate over properties of null and get a more cryptic error.

[ne-spal]

Oh, I got it.
And moreover, it's the same behaviour as the native implementaion.
But I thought it could be an exception for status 204...