Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use response._bodyInit as fallback for react-native and qq #398

Merged
merged 11 commits into from
Oct 8, 2024
Merged

fix: use response._bodyInit as fallback for react-native and qq #398

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
bcdc6da
add _bodyInit check for react native
joshmossas May 24, 2024
c50cb05
Merge branch 'main' of https://github.com/unjs/ofetch
joshmossas Jun 4, 2024
8a33ead
Merge branch 'main' of https://github.com/unjs/ofetch
joshmossas Jun 8, 2024
e2a372b
Merge branch 'main' of https://github.com/unjs/ofetch
joshmossas Jul 3, 2024
59ebff4
Merge branch 'main' of https://github.com/unjs/ofetch
joshmossas Aug 7, 2024
5dc0ba0
Merge branch 'main' of https://github.com/unjs/ofetch
joshmossas Oct 8, 2024
e11a6a0
Update fetch.ts
pi0 Oct 8, 2024
5ab0b94
check for _bodyInit in "stream" responseType case
joshmossas Oct 8, 2024
8ae2973
add ref to #294
pi0 Oct 8, 2024
df30e5d
Update fetch.ts
pi0 Oct 8, 2024
6c5317b
chore: apply automated updates
autofix-ci[bot] Oct 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions src/fetch.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,11 @@ export function createFetch(globalOptions: CreateFetchOptions = {}): $Fetch {
}

const hasBody =
context.response.body &&
(context.response.body ||
// https://github.com/unjs/ofetch/issues/324
// https://github.com/unjs/ofetch/issues/294
// https://github.com/JakeChampion/fetch/issues/1454
(context.response as any)._bodyInit) &&
joshmossas marked this conversation as resolved.
Show resolved Hide resolved
!nullBodyResponses.has(context.response.status) &&
context.options.method !== "HEAD";
if (hasBody) {
Expand All @@ -213,7 +217,8 @@ export function createFetch(globalOptions: CreateFetchOptions = {}): $Fetch {
break;
}
case "stream": {
context.response._data = context.response.body;
context.response._data =
context.response.body || (context.response as any)._bodyInit; // (see refs above)
break;
}
default: {
Expand Down