From 1652ac5735211c3bc6940a6a91ff7aba31f83a3d Mon Sep 17 00:00:00 2001 From: Joseph Frazier <1212jtraceur@gmail.com> Date: Sat, 24 Mar 2018 21:38:13 -0400 Subject: [PATCH] Use is-buffer instead of checking constructor This makes it work with browserified/webpacked Buffers (Uint8Array instances) --- lib/exif/ExifImage.js | 3 ++- package.json | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/exif/ExifImage.js b/lib/exif/ExifImage.js index 75fc4be..5428a4c 100644 --- a/lib/exif/ExifImage.js +++ b/lib/exif/ExifImage.js @@ -5,6 +5,7 @@ var assert = require('assert'); var fs = require('fs'); var util = require('util'); var BufferExtender = require('./Buffer'); // bad idea +var isBuffer = require('is-buffer'); var debug = require('debug')('exif'); var DEFAULT_MAX_ENTRIES=128; @@ -142,7 +143,7 @@ ExifImage.prototype.loadImage = function (image, callback) { debug("loadImage image=", image); - if (image.constructor.name === 'Buffer') { + if (isBuffer(image)) { this.processImage("Buffer", image, callback); return; } diff --git a/package.json b/package.json index 3de8dbb..32e9c6d 100644 --- a/package.json +++ b/package.json @@ -10,7 +10,8 @@ "url" : "http://github.com/gomfunkel/node-exif.git" }, "dependencies": { - "debug": "^2.2" + "debug": "^2.2", + "is-buffer": "^2.0.2" }, "devDependencies": { "mocha": "^2.4"