fetch-har 5.0.3 → 6.0.1

package/CHANGELOG.md

@@ -1,3 +1,72 @@
+## <small>6.0.1 (2022-01-26)</small>
+
+* fix: regression in query parameter handling (#242) ([6c6a147](https://github.com/readmeio/fetch-har/commit/6c6a147)), closes [#242](https://github.com/readmeio/fetch-har/issues/242)
+
+
+
+## <small>6.0.0 (2022-01-25)</small>
+
+> **BREAKING CHANGE**
+>
+> With this release we no longer support the [form-data](https://npm.im/form-data) package as it's handling of `multipart/form-data` is non-compliant with the browser `FormData` API. Please use [formdata-node](https://npm.im/formdata-node) or [formdata-polyfill](https://npm.im/formdata-polyfill) instead.
+>
+> Additionally the optional `userAgent` option has been reworked into a new options object that also supports a new `files` mapping array for augmenting binaries or file uploads within a HAR with a file buffers or the `File` API. See the readme for documentation.
+
+* fix: handling of multipart/form-data requests and how we interact with FormData ([#237](https://github.com/readmeio/fetch-har/pull/237))
+* chore(deps-dev): bumping node-fetch ([#240]((https://github.com/readmeio/fetch-har/pull/240))
+* fix: improper handling of query strings ([#239](https://github.com/readmeio/fetch-har/pull/239))
+* feat: extending the new files option to allow overriding raw binary payloads ([#238](https://github.com/readmeio/fetch-har/pull/238))
+
+
+
+
+## <small>5.0.5 (2022-01-03)</small>
+
+* chore: removing the publishConfig from the package file ([05df1e7](https://github.com/readmeio/fetch-har/commit/05df1e7))
+* chore(deps-dev): bump @readme/eslint-config from 8.0.2 to 8.1.1 (#235) ([f10d1dd](https://github.com/readmeio/fetch-har/commit/f10d1dd)), closes [#235](https://github.com/readmeio/fetch-har/issues/235)
+* chore(deps-dev): bump body-parser from 1.19.0 to 1.19.1 (#225) ([a42b49d](https://github.com/readmeio/fetch-har/commit/a42b49d)), closes [#225](https://github.com/readmeio/fetch-har/issues/225)
+* chore(deps-dev): bump eslint from 8.3.0 to 8.6.0 (#228) ([e592606](https://github.com/readmeio/fetch-har/commit/e592606)), closes [#228](https://github.com/readmeio/fetch-har/issues/228)
+* chore(deps-dev): bump har-examples from 2.0.2 to 2.1.0 (#231) ([efaa16e](https://github.com/readmeio/fetch-har/commit/efaa16e)), closes [#231](https://github.com/readmeio/fetch-har/issues/231)
+* chore(deps-dev): bump jest from 27.4.2 to 27.4.5 (#234) ([24f099e](https://github.com/readmeio/fetch-har/commit/24f099e)), closes [#234](https://github.com/readmeio/fetch-har/issues/234)
+* chore(deps-dev): bump jest-puppeteer from 6.0.2 to 6.0.3 (#229) ([fcd9262](https://github.com/readmeio/fetch-har/commit/fcd9262)), closes [#229](https://github.com/readmeio/fetch-har/issues/229)
+* chore(deps-dev): bump multer from 1.4.3 to 1.4.4 (#232) ([4119c42](https://github.com/readmeio/fetch-har/commit/4119c42)), closes [#232](https://github.com/readmeio/fetch-har/issues/232)
+* chore(deps-dev): bump prettier from 2.5.0 to 2.5.1 (#233) ([4f96fae](https://github.com/readmeio/fetch-har/commit/4f96fae)), closes [#233](https://github.com/readmeio/fetch-har/issues/233)
+* chore(deps-dev): bump webpack from 5.64.4 to 5.65.0 (#230) ([f06706b](https://github.com/readmeio/fetch-har/commit/f06706b)), closes [#230](https://github.com/readmeio/fetch-har/issues/230)
+* chore(deps-dev): bump webpack-dev-middleware from 5.2.2 to 5.3.0 (#227) ([f68d06c](https://github.com/readmeio/fetch-har/commit/f68d06c)), closes [#227](https://github.com/readmeio/fetch-har/issues/227)
+* chore(deps): bump actions/setup-node from 2.5.0 to 2.5.1 (#226) ([38fa52b](https://github.com/readmeio/fetch-har/commit/38fa52b)), closes [#226](https://github.com/readmeio/fetch-har/issues/226)
+
+
+
+## <small>5.0.4 (2021-12-01)</small>
+
+* chore(deps-dev): bump @readme/eslint-config from 7.1.1 to 7.2.2 (#211) ([1081874](https://github.com/readmeio/fetch-har/commit/1081874)), closes [#211](https://github.com/readmeio/fetch-har/issues/211)
+* chore(deps-dev): bump cookie-parser from 1.4.5 to 1.4.6 (#219) ([41d8c60](https://github.com/readmeio/fetch-har/commit/41d8c60)), closes [#219](https://github.com/readmeio/fetch-har/issues/219)
+* chore(deps-dev): bump eslint from 8.2.0 to 8.3.0 (#215) ([ab0e294](https://github.com/readmeio/fetch-har/commit/ab0e294)), closes [#215](https://github.com/readmeio/fetch-har/issues/215)
+* chore(deps-dev): bump har-examples from 2.0.1 to 2.0.2 (#204) ([87b1742](https://github.com/readmeio/fetch-har/commit/87b1742)), closes [#204](https://github.com/readmeio/fetch-har/issues/204)
+* chore(deps-dev): bump jest from 27.2.0 to 27.2.4 (#205) ([37446e8](https://github.com/readmeio/fetch-har/commit/37446e8)), closes [#205](https://github.com/readmeio/fetch-har/issues/205)
+* chore(deps-dev): bump jest from 27.2.4 to 27.3.1 (#212) ([bc85194](https://github.com/readmeio/fetch-har/commit/bc85194)), closes [#212](https://github.com/readmeio/fetch-har/issues/212)
+* chore(deps-dev): bump jest from 27.3.1 to 27.4.2 (#221) ([8c9dafd](https://github.com/readmeio/fetch-har/commit/8c9dafd)), closes [#221](https://github.com/readmeio/fetch-har/issues/221)
+* chore(deps-dev): bump jest-puppeteer from 5.0.4 to 6.0.0 (#207) ([11ccb44](https://github.com/readmeio/fetch-har/commit/11ccb44)), closes [#207](https://github.com/readmeio/fetch-har/issues/207)
+* chore(deps-dev): bump jest-puppeteer from 6.0.0 to 6.0.2 (#222) ([d17d217](https://github.com/readmeio/fetch-har/commit/d17d217)), closes [#222](https://github.com/readmeio/fetch-har/issues/222)
+* chore(deps-dev): bump nock from 13.1.3 to 13.1.4 (#209) ([4f81357](https://github.com/readmeio/fetch-har/commit/4f81357)), closes [#209](https://github.com/readmeio/fetch-har/issues/209)
+* chore(deps-dev): bump nock from 13.1.4 to 13.2.1 (#224) ([fefd729](https://github.com/readmeio/fetch-har/commit/fefd729)), closes [#224](https://github.com/readmeio/fetch-har/issues/224)
+* chore(deps-dev): bump node-fetch from 2.6.1 to 2.6.5 (#208) ([eadd607](https://github.com/readmeio/fetch-har/commit/eadd607)), closes [#208](https://github.com/readmeio/fetch-har/issues/208)
+* chore(deps-dev): bump node-fetch from 2.6.5 to 2.6.6 (#214) ([6bfe538](https://github.com/readmeio/fetch-har/commit/6bfe538)), closes [#214](https://github.com/readmeio/fetch-har/issues/214)
+* chore(deps-dev): bump prettier from 2.4.1 to 2.5.0 (#216) ([dee54f6](https://github.com/readmeio/fetch-har/commit/dee54f6)), closes [#216](https://github.com/readmeio/fetch-har/issues/216)
+* chore(deps-dev): bump webpack from 5.53.0 to 5.55.1 (#202) ([427332e](https://github.com/readmeio/fetch-har/commit/427332e)), closes [#202](https://github.com/readmeio/fetch-har/issues/202)
+* chore(deps-dev): bump webpack from 5.55.1 to 5.61.0 (#213) ([8276f5f](https://github.com/readmeio/fetch-har/commit/8276f5f)), closes [#213](https://github.com/readmeio/fetch-har/issues/213)
+* chore(deps-dev): bump webpack from 5.61.0 to 5.64.4 (#220) ([0dcfb6c](https://github.com/readmeio/fetch-har/commit/0dcfb6c)), closes [#220](https://github.com/readmeio/fetch-har/issues/220)
+* chore(deps-dev): bump webpack-dev-middleware from 5.1.0 to 5.2.1 (#206) ([94f0ad2](https://github.com/readmeio/fetch-har/commit/94f0ad2)), closes [#206](https://github.com/readmeio/fetch-har/issues/206)
+* chore(deps-dev): bump webpack-dev-middleware from 5.2.1 to 5.2.2 (#223) ([90c0501](https://github.com/readmeio/fetch-har/commit/90c0501)), closes [#223](https://github.com/readmeio/fetch-har/issues/223)
+* chore(deps-dev): upgrading eslint to v8 ([f06d433](https://github.com/readmeio/fetch-har/commit/f06d433))
+* chore(deps): bump actions/checkout from 2.3.4 to 2.3.5 (#210) ([5360ba9](https://github.com/readmeio/fetch-har/commit/5360ba9)), closes [#210](https://github.com/readmeio/fetch-har/issues/210)
+* chore(deps): bump actions/checkout from 2.3.5 to 2.4.0 (#217) ([0e46da0](https://github.com/readmeio/fetch-har/commit/0e46da0)), closes [#217](https://github.com/readmeio/fetch-har/issues/217)
+* chore(deps): bump actions/setup-node from 2.4.0 to 2.4.1 (#203) ([e6a4529](https://github.com/readmeio/fetch-har/commit/e6a4529)), closes [#203](https://github.com/readmeio/fetch-har/issues/203)
+* chore(deps): bump actions/setup-node from 2.4.1 to 2.5.0 (#218) ([e2cab24](https://github.com/readmeio/fetch-har/commit/e2cab24)), closes [#218](https://github.com/readmeio/fetch-har/issues/218)
+* docs: adding a code of conduct ([cbc9fa0](https://github.com/readmeio/fetch-har/commit/cbc9fa0))
+
+
+
 ## <small>5.0.3 (2021-09-16)</small>
 
 * ci: updating codeql to run tests against main ([f3afd13](https://github.com/readmeio/fetch-har/commit/f3afd13))

package/CODE_OF_CONDUCT.md

@@ -0,0 +1,128 @@
+# Contributor Covenant Code of Conduct
+
+## Our Pledge
+
+We as members, contributors, and leaders pledge to make participation in our
+community a harassment-free experience for everyone, regardless of age, body
+size, visible or invisible disability, ethnicity, sex characteristics, gender
+identity and expression, level of experience, education, socio-economic status,
+nationality, personal appearance, race, religion, or sexual identity
+and orientation.
+
+We pledge to act and interact in ways that contribute to an open, welcoming,
+diverse, inclusive, and healthy community.
+
+## Our Standards
+
+Examples of behavior that contributes to a positive environment for our
+community include:
+
+* Demonstrating empathy and kindness toward other people
+* Being respectful of differing opinions, viewpoints, and experiences
+* Giving and gracefully accepting constructive feedback
+* Accepting responsibility and apologizing to those affected by our mistakes,
+  and learning from the experience
+* Focusing on what is best not just for us as individuals, but for the
+  overall community
+
+Examples of unacceptable behavior include:
+
+* The use of sexualized language or imagery, and sexual attention or
+  advances of any kind
+* Trolling, insulting or derogatory comments, and personal or political attacks
+* Public or private harassment
+* Publishing others' private information, such as a physical or email
+  address, without their explicit permission
+* Other conduct which could reasonably be considered inappropriate in a
+  professional setting
+
+## Enforcement Responsibilities
+
+Community leaders are responsible for clarifying and enforcing our standards of
+acceptable behavior and will take appropriate and fair corrective action in
+response to any behavior that they deem inappropriate, threatening, offensive,
+or harmful.
+
+Community leaders have the right and responsibility to remove, edit, or reject
+comments, commits, code, wiki edits, issues, and other contributions that are
+not aligned to this Code of Conduct, and will communicate reasons for moderation
+decisions when appropriate.
+
+## Scope
+
+This Code of Conduct applies within all community spaces, and also applies when
+an individual is officially representing the community in public spaces.
+Examples of representing our community include using an official e-mail address,
+posting via an official social media account, or acting as an appointed
+representative at an online or offline event.
+
+## Enforcement
+
+Instances of abusive, harassing, or otherwise unacceptable behavior may be
+reported to the community leaders responsible for enforcement at
+[support+coc@readme.io](mailto:support+coc@readme.io).
+All complaints will be reviewed and investigated promptly and fairly.
+
+All community leaders are obligated to respect the privacy and security of the
+reporter of any incident.
+
+## Enforcement Guidelines
+
+Community leaders will follow these Community Impact Guidelines in determining
+the consequences for any action they deem in violation of this Code of Conduct:
+
+### 1. Correction
+
+**Community Impact**: Use of inappropriate language or other behavior deemed
+unprofessional or unwelcome in the community.
+
+**Consequence**: A private, written warning from community leaders, providing
+clarity around the nature of the violation and an explanation of why the
+behavior was inappropriate. A public apology may be requested.
+
+### 2. Warning
+
+**Community Impact**: A violation through a single incident or series
+of actions.
+
+**Consequence**: A warning with consequences for continued behavior. No
+interaction with the people involved, including unsolicited interaction with
+those enforcing the Code of Conduct, for a specified period of time. This
+includes avoiding interactions in community spaces as well as external channels
+like social media. Violating these terms may lead to a temporary or
+permanent ban.
+
+### 3. Temporary Ban
+
+**Community Impact**: A serious violation of community standards, including
+sustained inappropriate behavior.
+
+**Consequence**: A temporary ban from any sort of interaction or public
+communication with the community for a specified period of time. No public or
+private interaction with the people involved, including unsolicited interaction
+with those enforcing the Code of Conduct, is allowed during this period.
+Violating these terms may lead to a permanent ban.
+
+### 4. Permanent Ban
+
+**Community Impact**: Demonstrating a pattern of violation of community
+standards, including sustained inappropriate behavior,  harassment of an
+individual, or aggression toward or disparagement of classes of individuals.
+
+**Consequence**: A permanent ban from any sort of public interaction within
+the community.
+
+## Attribution
+
+This Code of Conduct is adapted from the [Contributor Covenant][homepage],
+version 2.0, available at
+https://www.contributor-covenant.org/version/2/0/code_of_conduct.html.
+
+Community Impact Guidelines were inspired by [Mozilla's code of conduct
+enforcement ladder](https://github.com/mozilla/diversity).
+
+[homepage]: https://www.contributor-covenant.org
+
+For answers to common questions about this code of conduct, see the FAQ at
+https://www.contributor-covenant.org/faq. Translations are available at
+https://www.contributor-covenant.org/translations.

package/README.md

@@ -13,13 +13,17 @@ npm install --save fetch-har
 
 ## Usage
 ```js
-const fetchHar = require('fetch-har');
+require('isomorphic-fetch');
+const fetchHar = require('.');
 
-// If executing from an environment without `fetch`, you'll need to polyfill.
-global.fetch = require('node-fetch');
-global.Headers = require('node-fetch').Headers;
-global.Request = require('node-fetch').Request;
-global.FormData = require('form-data');
+// If executing from an environment that dodoesn't normally provide fetch()
+// you'll need to polyfill some APIs in order to make `multipart/form-data`
+// requests.
+if (!globalThis.FormData) {
+  globalThis.Blob = require('formdata-node').Blob;
+  globalThis.File = require('formdata-node').File;
+  globalThis.FormData = require('formdata-node').FormData;
+}
 
 const har = {
   log: {
@@ -36,7 +40,10 @@ const har = {
               value: 'application/json',
             },
           ],
-          queryString: [{ name: 'a', value: 1 }, { name: 'b', value: 2 }],
+          queryString: [
+            { name: 'a', value: 1 },
+            { name: 'b', value: 2 },
+          ],
           postData: {
             mimeType: 'application/json',
             text: '{"id":8,"category":{"id":6,"name":"name"},"name":"name"}',
@@ -50,22 +57,48 @@ const har = {
 };
 
 fetchHar(har)
-  .then(request => request.json())
+  .then(res => res.json())
   .then(console.log);
 ```
 
-### `fetchHar(har, userAgent) => Promise`
+### API
+If you are executing `fetch-har` in a browser environment that supports the [FormData API](https://developer.mozilla.org/en-US/docs/Web/API/FormData) then you don't need to do anything. If you arent, however, you'll need to polyfill it.
+
+Unfortunately the most popular NPM package [form-data](https://npm.im/form-data) ships with a [non-spec compliant API](https://github.com/form-data/form-data/issues/124), and for this we don't recommend you use it, as if you use `fetch-har` to upload files it may not work.
+
+We recommend either [formdata-node](https://npm.im/formdata-node) or [formdata-polyfill](https://npm.im/formdata-polyfill).
+
+#### Options
+##### userAgent
+A custom `User-Agent` header to apply to your request. Please note that browsers have their own handling for these headers in `fetch()` calls so it may not work everywhere; it will always be sent in Node however.
+
+```js
+await fetchHar(har, { userAgent: 'my-client/1.0' });
+```
+
+##### files
+An optional object map you can supply to use for `multipart/form-data` file uploads in leu of relying on if the HAR you have has [data URLs](https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/Data_URIs). It supports Node file buffers and the [File](https://developer.mozilla.org/en-US/docs/Web/API/File) API.
+
+```js
+await fetchHar(har, { files: {
+  'owlbert.png': await fs.readFile('./owlbert.png'),
+  'file.txt': document.querySelector('#some-file-input').files[0],
+} });
+```
 
-- `har` is a [har](https://en.wikipedia.org/wiki/.har) file format.
-- `userAgent` is an optional user agent string to let you declare where the request is coming from.
+If you don't supply this option `fetch-har` will fallback to the data URL present within the supplied HAR. If no `files` option is present, and no data URL (via `param.value`) is present in the HAR, a fatal exception will be thrown.
 
-Performs a fetch request from a given HAR definition. HAR definitions can be used to list lots of requests but we only use the first from the `log.entries` array.
+##### multipartEncoder
+> ❗ If you are using `fetch-har` in Node you may need this option!
 
-### `fetchHar.constructRequest(har, userAgent) => Request`
+If you are running `fetch-har` within a Node environment and you're using `node-fetch@2`, or another `fetch` polyfill that does not support a spec-compliant `FormData` API, you will need to specify an encoder that will transform your `FormData` object into something that can be used with [Request.body](https://developer.mozilla.org/en-US/docs/Web/API/Request/body).
 
-- `har` is a [har](https://en.wikipedia.org/wiki/.har) file format.
-- `userAgent` is an optional user agent string to let you declare where the request is coming from.
+We recommend [form-data-encoder](https://npm.im/form-data-encoder).
 
-We also export a second function which is used to construct a [Request](https://developer.mozilla.org/en-US/docs/Web/API/Request) object from your HAR.
+```js
+const { FormDataEncoder } = require('form-data-encoder');
+
+await fetchHar(har, { multipartEncoder: FormDataEncoder });
+```
 
-This function is mainly exported for testing purposes but could be useful if you want to construct a request but do not want to execute it right away.
+You do **not**, and shouldn't, need to use this option in browser environments.

package/example.js

@@ -1,11 +1,14 @@
 /* eslint-disable import/no-extraneous-dependencies, no-console */
+require('isomorphic-fetch');
 const fetchHar = require('.');
 
-// If executing from an environment without `fetch`, you'll need to polyfill
-global.fetch = require('node-fetch');
-global.Headers = require('node-fetch').Headers;
-global.Request = require('node-fetch').Request;
-global.FormData = require('form-data');
+// If executing from an environment that dodoesn't normally provide fetch() you'll need to polyfill some APIs in order
+// to make `multipart/form-data` requests.
+if (!globalThis.FormData) {
+  globalThis.Blob = require('formdata-node').Blob;
+  globalThis.File = require('formdata-node').File;
+  globalThis.FormData = require('formdata-node').FormData;
+}
 
 const har = {
   log: {
@@ -39,5 +42,5 @@ const har = {
 };
 
 fetchHar(har)
-  .then(request => request.json())
+  .then(res => res.json())
   .then(console.log);

package/index.js

@@ -1,7 +1,76 @@
-/* eslint-disable no-case-declarations */
+const { Readable } = require('readable-stream');
 const parseDataUrl = require('parse-data-url');
 
-function constructRequest(har, userAgent = false) {
+if (!globalThis.Blob) {
+  try {
+    // eslint-disable-next-line import/no-extraneous-dependencies
+    globalThis.Blob = require('formdata-node').Blob;
+  } catch (e) {
+    throw new Error(
+      'Since you do not have the Blob API available in this environment you must install the optional `formdata-node` dependency.'
+    );
+  }
+}
+
+if (!globalThis.File) {
+  try {
+    // eslint-disable-next-line import/no-extraneous-dependencies
+    globalThis.File = require('formdata-node').File;
+  } catch (e) {
+    throw new Error(
+      'Since you do not have the File API available in this environment you must install the optional `formdata-node` dependency.'
+    );
+  }
+}
+
+function isBrowser() {
+  return typeof window !== 'undefined' && typeof document !== 'undefined';
+}
+
+function isBuffer(value) {
+  return typeof Buffer !== 'undefined' && Buffer.isBuffer(value);
+}
+
+function isFile(value) {
+  if (value instanceof File) {
+    // The `Blob` polyfill on Node comes back as being an instanceof `File`. Because passing a Blob into
+    // a File will end up with a corrupted file we want to prevent this.
+    //
+    // This object identity crisis does not happen in the browser.
+    return value.constructor.name === 'File';
+  }
+
+  return false;
+}
+
+/**
+ * @license MIT
+ * @see {@link https://github.com/octet-stream/form-data-encoder/blob/master/lib/util/isFunction.ts}
+ */
+function isFunction(value) {
+  return typeof value === 'function';
+}
+
+/**
+ * We're loading this library in here instead of loading it from `form-data-encoder` because that uses lookbehind
+ * regex in its main encoder that Safari doesn't support so it throws a fatal page exception.
+ *
+ * @license MIT
+ * @see {@link https://github.com/octet-stream/form-data-encoder/blob/master/lib/util/isFormData.ts}
+ */
+function isFormData(value) {
+  return (
+    value &&
+    isFunction(value.constructor) &&
+    value[Symbol.toStringTag] === 'FormData' && // eslint-disable-line compat/compat
+    isFunction(value.append) &&
+    isFunction(value.getAll) &&
+    isFunction(value.entries) &&
+    isFunction(value[Symbol.iterator]) // eslint-disable-line compat/compat
+  );
+}
+
+function constructRequest(har, opts = { userAgent: false, files: false, multipartEncoder: false }) {
   if (!har) throw new Error('Missing HAR definition');
   if (!har.log || !har.log.entries || !har.log.entries.length) throw new Error('Missing log.entries array');
 
@@ -31,7 +100,7 @@ function constructRequest(har, userAgent = false) {
     // As the browser fetch API can't set custom cookies for requests, they instead need to be defined on the document
     // and passed into the request via `credentials: include`. Since this is a browser-specific quirk, that should only
     // happen in browsers!
-    if (typeof window !== 'undefined' && typeof document !== 'undefined') {
+    if (isBrowser()) {
       request.cookies.forEach(cookie => {
         document.cookie = `${encodeURIComponent(cookie.name)}=${encodeURIComponent(cookie.value)}`;
       });
@@ -68,7 +137,10 @@ function constructRequest(har, userAgent = false) {
           options.body = encodedParams.toString();
           break;
 
+        case 'multipart/alternative':
         case 'multipart/form-data':
+        case 'multipart/mixed':
+        case 'multipart/related':
           // If there's a Content-Type header set remove it. We're doing this because when we pass the form data object
           // into `fetch` that'll set a proper `Content-Type` header for this request that also includes the boundary
           // used on the content.
@@ -79,59 +151,93 @@ function constructRequest(har, userAgent = false) {
             headers.delete('Content-Type');
           }
 
-          // The `form-data` NPM module returns one of two things: a native `FormData` API, or its own polyfill. Since
-          // the polyfill does not support the full API of the native FormData object, when you load `form-data` within
-          // a browser environment you'll have two major differences in API:
-          //
-          //  * The `.append()` API in `form-data` requires that the third argument is an object containing various,
-          //    undocumented, options. In the browser, `.append()`'s third argument should only be present when the
-          //    second is a `Blob` or `USVString`, and when it is present, it should be a filename string.
-          //  * `form-data` does not expose an `.entries()` API, so the only way to retrieve data out of it for
-          //    construction of boundary-separated payload content is to use its `.pipe()` API. Since the browser
-          //    doesn't have this API, you'll be unable to retrieve data out of it.
-          //
-          // Now since the native `FormData` API is iterable, and has the `.entries()` iterator, we can easily detect
-          // what version of the FormData API we have access to by looking for this and constructing a simple wrapper
-          // to disconnect some of this logic so you can work against a single, consistent API.
-          //
-          // Having to do this isn't fun, but it's the only way you can write code to work with `multipart/form-data`
-          // content under a server and browser.
           const form = new FormData();
-          const isNativeFormData = typeof form[Symbol.iterator] === 'function';
+          if (!isFormData(form)) {
+            // The `form-data` NPM module returns one of two things: a native `FormData` API or its own polyfill.
+            // Unfortunately this polyfill does not support the full API of the native FormData object so when you load
+            // `form-data` within a browser environment you'll have two major differences in API:
+            //
+            //  * The `.append()` API in `form-data` requires that the third argument is an object containing various,
+            //    undocumented, options. In the browser, `.append()`'s third argument should only be present when the
+            //    second is a `Blob` or `USVString`, and when it is present, it should be a filename string.
+            //  * `form-data` does not expose an `.entries()` API, so the only way to retrieve data out of it for
+            //    construction of boundary-separated payload content is to use its `.pipe()` API. Since the browser
+            //    doesn't have this API, you'll be unable to retrieve data out of it.
+            //
+            // Now since the native `FormData` API is iterable, and has the `.entries()` iterator, we can easily detect
+            // if we have a native copy of the FormData API. It's for all of these reasons that we're opting to hard
+            // crash here because supporting this non-compliant API is more trouble than its worth.
+            //
+            // https://github.com/form-data/form-data/issues/124
+            throw new Error(
+              "We've detected you're using a non-spec compliant FormData library. We recommend polyfilling FormData with https://npm.im/formdata-node"
+            );
+          }
 
           request.postData.params.forEach(param => {
-            if ('fileName' in param && !('value' in param)) {
-              throw new Error(
-                "The supplied HAR has a postData parameter with `fileName`, but no `value` content. Since this library doesn't have access to the filesystem, it can't fetch that file."
-              );
-            }
-
-            // If the incoming parameter is a file, and that files value is a data URL, we should decode that and set
-            // the contents of the value in the HAR to the actual contents of the file.
             if ('fileName' in param) {
-              const parsed = parseDataUrl(param.value);
-              if (parsed) {
-                // eslint-disable-next-line no-param-reassign
-                param.value = parsed.toBuffer().toString();
-              }
-            }
+              if (opts.files && param.fileName in opts.files) {
+                const fileContents = opts.files[param.fileName];
+
+                // If the file we've got available to us is a Buffer then we need to convert it so that the FormData
+                // API can use it.
+                if (isBuffer(fileContents)) {
+                  form.set(
+                    param.name,
+                    new File([fileContents], param.fileName, {
+                      type: param.contentType || null,
+                    }),
+                    param.fileName
+                  );
+
+                  return;
+                } else if (isFile(fileContents)) {
+                  form.set(param.name, fileContents, param.fileName);
+                  return;
+                }
+
+                throw new TypeError(
+                  'An unknown object has been supplied into the `files` config for use. We only support instances of the File API and Node Buffer objects.'
+                );
+              } else if ('value' in param) {
+                let paramBlob;
+                const parsed = parseDataUrl(param.value);
+                if (parsed) {
+                  // If we were able to parse out this data URL we don't need to transform its data into a buffer for
+                  // `Blob` because that supports data URLs already.
+                  paramBlob = new Blob([param.value], { type: parsed.contentType || param.contentType || null });
+                } else {
+                  paramBlob = new Blob([param.value], { type: param.contentType || null });
+                }
 
-            if (isNativeFormData) {
-              if ('fileName' in param) {
-                const paramBlob = new Blob([param.value], { type: param.contentType || null });
                 form.append(param.name, paramBlob, param.fileName);
-              } else {
-                form.append(param.name, param.value);
+                return;
               }
-            } else {
-              form.append(param.name, param.value || '', {
-                filename: param.fileName || null,
-                contentType: param.contentType || null,
-              });
+
+              throw new Error(
+                "The supplied HAR has a postData parameter with `fileName`, but neither `value` content within the HAR or any file buffers were supplied with the `files` option. Since this library doesn't have access to the filesystem, it can't fetch that file."
+              );
             }
+
+            form.append(param.name, param.value);
           });
 
-          options.body = form;
+          // If a the `fetch` polyfill that's being used here doesn't have spec-compliant handling for the `FormData`
+          // API (like `node-fetch@2`), then you should pass in a handler (like the `form-data-encoder` library) to
+          // transform its contents into something that can be used with the `Request` object.
+          //
+          // https://www.npmjs.com/package/formdata-node
+          if (opts.multipartEncoder) {
+            // eslint-disable-next-line new-cap
+            const encoder = new opts.multipartEncoder(form);
+            Object.keys(encoder.headers).forEach(header => {
+              headers.set(header, encoder.headers[header]);
+            });
+
+            options.body = Readable.from(encoder);
+          } else {
+            options.body = form;
+          }
           break;
 
         default:
@@ -148,27 +254,62 @@ function constructRequest(har, userAgent = false) {
 
           options.body = JSON.stringify(formBody);
       }
-    } else {
-      options.body = request.postData.text;
+    } else if (request.postData.text.length) {
+      // If we've got `files` map content present, and this post data content contains a valid data URL then we can
+      // substitute the payload with that file instead of the using data URL.
+      if (opts.files) {
+        const parsed = parseDataUrl(request.postData.text);
+        if (parsed && 'name' in parsed && parsed.name in opts.files) {
+          const fileContents = opts.files[parsed.name];
+          if (isBuffer(fileContents)) {
+            options.body = fileContents;
+          } else if (isFile(fileContents)) {
+            // `Readable.from` isn't available in browsers but the browser `Request` object can handle `File` objects
+            // just fine without us having to mold it into shape.
+            if (isBrowser()) {
+              options.body = fileContents;
+            } else {
+              options.body = Readable.from(fileContents.stream());
+
+              // Supplying a polyfilled `File` stream into `Request.body` doesn't automatically add `Content-Length`.
+              if (!headers.has('content-length')) {
+                headers.set('content-length', fileContents.size);
+              }
+            }
+          }
+        }
+      }
+
+      if (typeof options.body === 'undefined') {
+        options.body = request.postData.text;
+      }
     }
   }
 
+  // We automaticaly assume that the HAR that we have already has query parameters encoded within it so we do **not**
+  // use the `URLSearchParams` API here for composing the query string.
   if ('queryString' in request && request.queryString.length) {
-    const query = request.queryString.map(q => `${q.name}=${q.value}`).join('&');
-    querystring = `?${query}`;
+    const urlObj = new URL(url);
+
+    const queryParams = Array.from(urlObj.searchParams).map(([k, v]) => `${k}=${v}`);
+    request.queryString.forEach(q => {
+      queryParams.push(`${q.name}=${q.value}`);
+    });
+
+    querystring = queryParams.join('&');
   }
 
-  if (userAgent) {
-    headers.append('User-Agent', userAgent);
+  if (opts.userAgent) {
+    headers.append('User-Agent', opts.userAgent);
   }
 
   options.headers = headers;
 
-  return new Request(`${url}${querystring}`, options);
+  return new Request(`${url.split('?')[0]}${querystring ? `?${querystring}` : ''}`, options);
 }
 
-function fetchHar(har, userAgent) {
-  return fetch(constructRequest(har, userAgent));
+function fetchHar(har, opts = { userAgent: false, files: false, multipartEncoder: false }) {
+  return fetch(constructRequest(har, opts));
 }
 
 module.exports = fetchHar;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fetch-har",
-  "version": "5.0.3",
+  "version": "6.0.1",
   "description": "Make a fetch request from a HAR definition",
   "main": "index.js",
   "engines": {
@@ -12,10 +12,10 @@
     "prettier": "prettier --list-different --write \"./**.js\"",
     "release": "npx conventional-changelog-cli -i CHANGELOG.md -s",
     "serve": "node __tests__/server.js",
-    "test": "jest --coverage"
-  },
-  "publishConfig": {
-    "registry": "http://registry.npmjs.org"
+    "test:browser": "karma start --single-run",
+    "test:browser:chrome": "karma start --browsers=Chrome --single-run=false",
+    "test:browser:debug": "karma start --single-run=false",
+    "test": "nyc mocha"
   },
   "repository": {
     "type": "git",
@@ -27,37 +27,32 @@
   },
   "homepage": "https://github.com/readmeio/fetch-har#readme",
   "dependencies": {
-    "parse-data-url": "^4.0.1"
+    "parse-data-url": "^4.0.1",
+    "readable-stream": "^3.6.0"
+  },
+  "optionalDependencies": {
+    "fromdata-node": "^4.3.2"
   },
   "devDependencies": {
-    "@readme/eslint-config": "^7.1.1",
-    "body-parser": "^1.19.0",
-    "cookie-parser": "^1.4.5",
-    "eslint": "^7.31.0",
-    "eslint-plugin-compat": "^3.11.1",
-    "express": "^4.17.1",
+    "@jsdevtools/host-environment": "^2.1.2",
+    "@jsdevtools/karma-config": "^3.1.7",
+    "@readme/eslint-config": "^8.1.2",
+    "chai": "^4.3.4",
+    "eslint": "^8.7.0",
+    "eslint-plugin-compat": "^4.0.1",
+    "eslint-plugin-mocha": "^10.0.3",
     "form-data": "^4.0.0",
-    "har-examples": "^2.0.1",
-    "jest": "^27.2.0",
-    "jest-puppeteer": "^5.0.4",
-    "multer": "^1.4.2",
-    "nock": "^13.1.1",
+    "form-data-encoder": "^1.7.1",
+    "formdata-node": "^4.3.2",
+    "har-examples": "^3.1.0",
+    "isomorphic-fetch": "^3.0.0",
+    "mocha": "^9.1.4",
     "node-fetch": "^2.6.0",
-    "prettier": "^2.4.1",
-    "webpack": "^5.53.0",
-    "webpack-dev-middleware": "^5.1.0"
+    "nyc": "^15.1.0",
+    "prettier": "^2.5.1"
   },
   "browserslist": [
     "last 2 versions"
   ],
-  "prettier": "@readme/eslint-config/prettier",
-  "jest": {
-    "preset": "jest-puppeteer",
-    "globals": {
-      "SERVER_URL": "http://localhost:4444"
-    },
-    "testMatch": [
-      "<rootDir>/__tests__/*.test.js"
-    ]
-  }
+  "prettier": "@readme/eslint-config/prettier"
 }

package/jest-puppeteer.config.js

@@ -1,6 +0,0 @@
-module.exports = {
-  server: {
-    command: 'npm run serve',
-    port: 4444,
-  },
-};