Skip to content

Fix fetch erros - #1138

Open
solo-samurai wants to merge 3 commits into
dropbox:mainfrom
solo-samurai:fix-fetch-erros
Open

Fix fetch erros#1138
solo-samurai wants to merge 3 commits into
dropbox:mainfrom
solo-samurai:fix-fetch-erros

Conversation

@solo-samurai

@solo-samurai solo-samurai commented Apr 22, 2024

Copy link
Copy Markdown

Upgrade body access: Uses body.arrayBuffer() for response body, improving compatibility with future Node.js versions.
Load node-fetch: Only requires node-fetch when needed fixing node warnings.

fixes: #1137, #1135, #1130

Checklist

General Contributing

  • Have you read the Code of Conduct and signed the CLA?

Is This a Code Change?

  • Non-code related change (markdown/git settings etc)
  • SDK Code Change
  • Example/Test Code Change

Validation

  • Does npm test pass?
  • Does npm run build pass?
  • Does npm run lint pass?

@greg-db

greg-db commented Apr 23, 2024

Copy link
Copy Markdown
Contributor

Thanks for putting this together! I'll ask the team to review this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change deprecated .buffer() to .arrayBuffer()

2 participants