-
Notifications
You must be signed in to change notification settings - Fork 124
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.
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: this param for fetch calls #2292
Conversation
@@ -53,7 +53,7 @@ | |||
"@rollup/plugin-json": "^4.1.0", | |||
"@rollup/plugin-node-resolve": "^13.1.3", | |||
"@types/mocha": "9.0.0", | |||
"bundlesize": "^0.18.1", | |||
"body-parser": "^1.20.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...was a missing dependency
headers = { ...headers, 'X-Name': encodeURIComponent(name) } | ||
const headers = { | ||
...Web3Storage.headers(token), | ||
'Content-Type': 'application/vnd.ipld.car', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ensures we always get a Content-Type
header even when we do the Node.js fix below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: I have created ipld/js-car#142 that we could reference official mime type, instead of having to copy that string all over.
"test:all": "run-s test:web test:cjs test:esm test:size", | ||
"test:size": "bundlesize", | ||
"test:all": "run-s test:web test:cjs test:esm", | ||
"test:size": "npx bundlesize", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
npx
so that npm install
in ndoe.js 20 doesn't fail because native module can't be compiled.
🤖 I have created a release *beep* *boop* --- ## [7.19.0](api-v7.18.1...api-v7.19.0) (2023-07-31) ### Features * write content claims ([#2291](#2291)) ([6a31c9b](6a31c9b)) ### Bug Fixes * remove unnecessary compatibility flags ([#2301](#2301)) ([436cdb8](436cdb8)) * signer DID ([82c3462](82c3462)) * this param for fetch calls ([#2292](#2292)) ([bf32c53](bf32c53)) * use error message value in cause ([9b07288](9b07288)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@web-std/fetch
tov4
with fix forthis
paramArrayBuffer
asfetch
body instead of aBlob
127.0.0.1
instead oflocalhost
so node doesn't auto resolve a IPv6 address and fail to connectnpm test
tasks since it cannot be run on node.js 20 (depends on unmaintained native moduleiltorb
which does not compile on node.js 20)resolves #2274