richardgirges / express-fileupload / 300

pending completion

Build # 300

Build Type

Pull #233

travis-ci-com

Committed by

web-flow

Commit Message

<a href="https://github.com/richardgirges/express-fileupload/commit/<a class=hub.com/richardgirges/express-fileupload/commit/<a class="double-link" href="https://git"><a class=hub.com/richardgirges/express-fileupload/commit/<a class="double-link" href="https://git"><a class=hub.com/richardgirges/express-fileupload/commit/<a class="double-link" href="https://git"><a class=hub.com/richardgirges/express-fileupload/commit/4e9ffdd6fe714d9d381049fd3ed9a77c997c2548">4e9ffdd6f">&lt;a href=&quot;https://github.com/richardgirges/express-fileupload/commit/</a><a class="double-link" href="https://github.com/richardgirges/express-fileupload/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/richardgirges/express-fileupload/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/richardgirges/express-fileupload/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/richardgirges/express-fileupload/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/richardgirges/express-fileupload/commit/4e9ffdd6fe714d9d381049fd3ed9a77c997c2548">4e9ffdd6f</a><a href="https://github.com/richardgirges/express-fileupload/commit/4e9ffdd6fe714d9d381049fd3ed9a77c997c2548">&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/richardgirges/express-fileupload/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/richardgirges/express-fileupload/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/richardgirges/express-fileupload/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/richardgirges/express-fileupload/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/richardgirges/express-fileupload/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/richardgirges/express-fileupload/commit/4e9ffdd6fe714d9d381049fd3ed9a77c997c2548&quot;&gt;4e9ffdd6f&lt;/a&gt;&lt;a href=&quot;https://github.com/richardgirges/express-fileupload/commit/4e9ffdd6fe714d9d381049fd3ed9a77c997c2548&quot;&gt;&amp;quot... (continued)

Pull Request Pull Request #233: Support empty files

Run Details

119 of 128 branches covered (92.97%)

26 of 26 new or added lines in 3 files covered. (100.0%)

269 of 281 relevant lines covered (95.73%)

2981.55 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

88.64

/lib/processMultipart.js

const Busboy = require('busboy');
const UploadTimer = require('./uploadtimer');
const fileFactory = require('./fileFactory');
const memHandler = require('./memHandler');
const tempFileHandler = require('./tempFileHandler');
const processNested = require('./processNested');
const {
  isFunc,
  debugLog,
  buildFields,
  buildOptions,
  parseFileName
} = require('./utilities');

const waitFlushProperty = Symbol('wait flush property symbol');

/**
 * Processes multipart request
 * Builds a req.body object for fields
 * Builds a req.files object for files
 * @param  {Object}   options expressFileupload and Busboy options
 * @param  {Object}   req     Express request object
 * @param  {Object}   res     Express response object
 * @param  {Function} next    Express next method
 * @return {void}
 */
module.exports = (options, req, res, next) => {
  req.files = null;

  // Build busboy options and init busboy instance.
  const busboyOptions = buildOptions(options, { headers: req.headers });
  const busboy = new Busboy(busboyOptions);

  // Close connection with specified reason and http code, default: 400 Bad Request.
  const closeConnection = (code, reason) => {
    req.unpipe(busboy);
    res.writeHead(code || 400, { Connection: 'close' });
    res.end(reason || 'Bad Request');
  };

  // Build multipart req.body fields
  busboy.on('field', (field, val) => req.body = buildFields(req.body, field, val));

  // Build req.files fields
  busboy.on('file', (field, file, name, encoding, mime) => {
    // Parse file name(cutting huge names, decoding, etc..).
    const filename = parseFileName(options, name);
    // Define methods and handlers for upload process.
    const {
      dataHandler,
      getFilePath,
      getFileSize,
      getHash,
      complete,
      cleanup,
      getWritePromise
    } = options.useTempFiles
      ? tempFileHandler(options, field, filename) // Upload into temporary file.
      : memHandler(options, field, filename);     // Upload into RAM.
    // Define upload timer.
    const uploadTimer = new UploadTimer(options.uploadTimeout, () => {
      file.removeAllListeners('data');
      file.resume();
      // After destroy an error event will be emitted and file clean up will be done.
      file.destroy(new Error(`Upload timeout ${field}->${filename}, bytes:${getFileSize()}`));
    });

    file.on('limit', () => {
      debugLog(options, `Size limit reached for ${field}->${filename}, bytes:${getFileSize()}`);
      // Reset upload timer in case of file limit reached.
      uploadTimer.clear();
      // Run a user defined limit handler if it has been set.
      if (isFunc(options.limitHandler)) return options.limitHandler(req, res, next);
      // Close connection with 413 code and do cleanup if abortOnLimit set(default: false).
      if (options.abortOnLimit) {
        debugLog(options, `Aborting upload because of size limit ${field}->${filename}.`);
        closeConnection(413, options.responseOnLimit);
        cleanup();
      }
    });

    file.on('data', (data) => {
      uploadTimer.set(); // Refresh upload timer each time new data chunk came.
      dataHandler(data); // Handle new piece of data.
    });

    file.on('end', () => {
      const size = getFileSize();
      // Debug logging for file upload ending.
      debugLog(options, `Upload finished ${field}->${filename}, bytes:${size}`);
      // Reset upload timer in case of end event.
      uploadTimer.clear();
      // See https://github.com/richardgirges/express-fileupload/issues/191
      // Do not add file instance to the req.files if original name and size are empty.
      // Empty name and zero size indicates empty file field in the posted form.
      if (!name && size === 0) {
        return debugLog(options, `Don't add file instance if original name and size are empty`);
      }
      req.files = buildFields(req.files, field, fileFactory({
        buffer: complete(),
        name: filename,
        tempFilePath: getFilePath(),
        hash: getHash(),
        size,
        encoding,
        truncated: file.truncated,
        mimetype: mime
      }, options));

      if (!req[waitFlushProperty]) {
        req[waitFlushProperty] = [];
      }
      req[waitFlushProperty].push(getWritePromise());
    });

    file.on('error', (err) => {
      uploadTimer.clear(); // Reset upload timer in case of errors.
      debugLog(options, err);
      cleanup();
      next();
    });

    // Debug logging for a new file upload.
    debugLog(options, `New upload started ${field}->${filename}, bytes:${getFileSize()}`);
    // Set new upload timeout for a new file.
    uploadTimer.set();
  });

  busboy.on('finish', () => {
    debugLog(options, `Busboy finished parsing request.`);
    if (options.parseNested) {
      req.body = processNested(req.body);
      req.files = processNested(req.files);
    }

    if (!req[waitFlushProperty]) return next();
    Promise.all(req[waitFlushProperty])
      .then(() => {
        delete req[waitFlushProperty];
        next();
      }).catch(err => {
        delete req[waitFlushProperty];
        debugLog(options, `Error while waiting files flush: ${err}`);
        next(err);
      });
  });

  busboy.on('error', (err) => {
    debugLog(options, `Busboy error`);
    next(err);
  });

  req.pipe(busboy);
};

1	const Busboy = require('busboy');	3✔
2	const UploadTimer = require('./uploadtimer');	3✔
3	const fileFactory = require('./fileFactory');	3✔
4	const memHandler = require('./memHandler');	3✔
5	const tempFileHandler = require('./tempFileHandler');	3✔
6	const processNested = require('./processNested');	3✔
7	const {	3✔
8	isFunc,
9	debugLog,
10	buildFields,
11	buildOptions,
12	parseFileName
13	} = require('./utilities');
14
15	const waitFlushProperty = Symbol('wait flush property symbol');	3✔
16
17	/**
18	* Processes multipart request
19	* Builds a req.body object for fields
20	* Builds a req.files object for files
21	* @param {Object} options expressFileupload and Busboy options
22	* @param {Object} req Express request object
23	* @param {Object} res Express response object
24	* @param {Function} next Express next method
25	* @return {void}
26	*/
27	module.exports = (options, req, res, next) => {	3✔
28	req.files = null;	258✔
29
30	// Build busboy options and init busboy instance.
31	const busboyOptions = buildOptions(options, { headers: req.headers });	258✔
32	const busboy = new Busboy(busboyOptions);	258✔
33
34	// Close connection with specified reason and http code, default: 400 Bad Request.
35	const closeConnection = (code, reason) => {	258✔
36	req.unpipe(busboy);	3✔
37	res.writeHead(code \|\| 400, { Connection: 'close' });	3!
38	res.end(reason \|\| 'Bad Request');	3!
39	};
40
41	// Build multipart req.body fields
42	busboy.on('field', (field, val) => req.body = buildFields(req.body, field, val));	258✔
43
44	// Build req.files fields
45	busboy.on('file', (field, file, name, encoding, mime) => {	258✔
46	// Parse file name(cutting huge names, decoding, etc..).
47	const filename = parseFileName(options, name);	261✔
48	// Define methods and handlers for upload process.
49	const {	261✔
50	dataHandler,
51	getFilePath,
52	getFileSize,
53	getHash,
54	complete,
55	cleanup,
56	getWritePromise
57	} = options.useTempFiles	261✔
58	? tempFileHandler(options, field, filename) // Upload into temporary file.
59	: memHandler(options, field, filename); // Upload into RAM.
60	// Define upload timer.
61	const uploadTimer = new UploadTimer(options.uploadTimeout, () => {	261✔
62	file.removeAllListeners('data');	9✔
63	file.resume();	9✔
64	// After destroy an error event will be emitted and file clean up will be done.
65	file.destroy(new Error(`Upload timeout ${field}->${filename}, bytes:${getFileSize()}`));	9✔
66	});
67
68	file.on('limit', () => {	261✔
69	debugLog(options, `Size limit reached for ${field}->${filename}, bytes:${getFileSize()}`);	9✔
70	// Reset upload timer in case of file limit reached.
71	uploadTimer.clear();	9✔
72	// Run a user defined limit handler if it has been set.
73	if (isFunc(options.limitHandler)) return options.limitHandler(req, res, next);	9✔
74	// Close connection with 413 code and do cleanup if abortOnLimit set(default: false).
75	if (options.abortOnLimit) {	6✔
76	debugLog(options, `Aborting upload because of size limit ${field}->${filename}.`);	3✔
77	closeConnection(413, options.responseOnLimit);	3✔
78	cleanup();	3✔
79	}
80	});
81
82	file.on('data', (data) => {	261✔
83	uploadTimer.set(); // Refresh upload timer each time new data chunk came.	984✔
84	dataHandler(data); // Handle new piece of data.	984✔
85	});
86
87	file.on('end', () => {	261✔
88	const size = getFileSize();	246✔
89	// Debug logging for file upload ending.
90	debugLog(options, `Upload finished ${field}->${filename}, bytes:${size}`);	246✔
91	// Reset upload timer in case of end event.
92	uploadTimer.clear();	246✔
93	// See https://github.com/richardgirges/express-fileupload/issues/191
94	// Do not add file instance to the req.files if original name and size are empty.
95	// Empty name and zero size indicates empty file field in the posted form.
96	if (!name && size === 0) {	246!
97	return debugLog(options, `Don't add file instance if original name and size are empty`);	×
98	}
99	req.files = buildFields(req.files, field, fileFactory({	246✔
100	buffer: complete(),
101	name: filename,
102	tempFilePath: getFilePath(),
103	hash: getHash(),
104	size,
105	encoding,
106	truncated: file.truncated,
107	mimetype: mime
108	}, options));
109
110	if (!req[waitFlushProperty]) {	246✔
111	req[waitFlushProperty] = [];	228✔
112	}
113	req[waitFlushProperty].push(getWritePromise());	246✔
114	});
115
116	file.on('error', (err) => {	261✔
117	uploadTimer.clear(); // Reset upload timer in case of errors.	9✔
118	debugLog(options, err);	9✔
119	cleanup();	9✔
120	next();	9✔
121	});
122
123	// Debug logging for a new file upload.
124	debugLog(options, `New upload started ${field}->${filename}, bytes:${getFileSize()}`);	261✔
125	// Set new upload timeout for a new file.
126	uploadTimer.set();	261✔
127	});
128
129	busboy.on('finish', () => {	258✔
130	debugLog(options, `Busboy finished parsing request.`);	243✔
131	if (options.parseNested) {	243✔
132	req.body = processNested(req.body);	3✔
133	req.files = processNested(req.files);	3✔
134	}
135
136	if (!req[waitFlushProperty]) return next();	243✔
137	Promise.all(req[waitFlushProperty])	228✔
138	.then(() => {
139	delete req[waitFlushProperty];	228✔
140	next();	228✔
141	}).catch(err => {
142	delete req[waitFlushProperty];	×
143	debugLog(options, `Error while waiting files flush: ${err}`);	×
144	next(err);	×
145	});
146	});
147
148	busboy.on('error', (err) => {	258✔
149	debugLog(options, `Busboy error`);	×
150	next(err);	×
151	});
152
153	req.pipe(busboy);	258✔
154	};

richardgirges / express-fileupload / 300

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous