UnhandledPromiseRejectionWarning caused by resolveBuffer on empty resource body (#886)

* Fix for UnhandledPromiseRejectionWarning in resolveBuffer

* code simplification

* reject with TypeError if body is not a Buffer
This commit is contained in:
Steffen
2016-11-13 08:07:51 +01:00
committed by Schuyler Cebulskie
parent c041b1bc23
commit a359f344d8

View File

@@ -244,7 +244,11 @@ class ClientDataResolver {
if (/^https?:\/\//.test(resource)) {
request.get(resource)
.set('Content-Type', 'blob')
.end((err, res) => err ? reject(err) : resolve(res.body));
.end((err, res) => {
if (err) return reject(err);
if (!(res.body instanceof Buffer)) return reject(new TypeError('Body is not a Buffer'));
return resolve(res.body);
});
} else {
const file = path.resolve(resource);
fs.stat(file, (err, stats) => {