fix: Add a condition to verify that the accept
of the request header starts with 'text'
#557
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed #556
[Example]
[Error Message]
Same as when use
JSON.parse('OK')
[Temporary solution]
[Description]
For
json
type, if there is no JSON type of response, an exception occurs when invoking theJSON.parse
function.The
text
method ofResponse
has been modified to return a string as it is in the response.I think we can discuss which values we should return (whether we return empty strings like 204)
I think it's an error to have a parsing error when the user didn't directly call the
json
method.2024.01.17
It was modified from using the try catch block to confirm the
accpet
of the request header.If the type is
json
and you can't parse it with JSON and you don't want to process it, I'd like your opinion.