Merge pull request #785 from stripe/jlomas/fix-file-create

Fix file uploads with nested params
stripe · Jan 30, 2020 · a6181ec · a6181ec
2 parents 8f6b0f8 + 9913bdf
commit a6181ec
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 4 deletions.
diff --git a/lib/multipart.js b/lib/multipart.js
@@ -28,8 +28,10 @@ const multipartDataGenerator = (method, data, headers) => {
  return `"${s.replace(/"|"/g, '%22').replace(/\r\n|\r|\n/g, ' ')}"`;
  }
 
- for (const k in utils.flattenAndStringify(data)) {
- const v = data[k];
+ const flattenedData = utils.flattenAndStringify(data);
+
+ for (const k in flattenedData) {
+ const v = flattenedData[k];
  push(`--${segno}`);
  if (v.hasOwnProperty('data')) {
  push(

diff --git a/testUtils/index.js b/testUtils/index.js
@@ -56,8 +56,22 @@ const utils = (module.exports = {
  if (host) {
  req.host = host;
  }
- stripeInstance.REQUESTS.push(req);
- cb.call(this, null, {});
+
+ const handleMockRequest = (err, req) => {
+ stripeInstance.REQUESTS.push(req);
+ cb.call(this, err, {});
+ };
+
+ if (this.requestDataProcessor) {
+ this.requestDataProcessor(
+ method,
+ data,
+ options.headers,
+ handleMockRequest
+ );
+ } else {
+ handleMockRequest(null, req);
+ }
  };
  }