node-metainspector icon indicating copy to clipboard operation
node-metainspector copied to clipboard

Big GIFs can cause application crash

Open lorenzos opened this issue 9 years ago • 5 comments

I don't know if all big files, or only specific ones, but this:

var mi = require('node-metainspector');
new mi("http://media.giphy.com/media/euBj15T6nrp6g/giphy.gif", {}).fetch();

Crashes the application with:

node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:47
  _.each(dom, function(elem) {
                      ^
RangeError: Maximum call stack size exceeded
    at /home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:47:23
    at Array.forEach (native)
    at Function._.each._.forEach (/home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/node_modules/underscore/underscore.js:78:11)
    at exports.connect (/home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:47:5)
    at /home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:64:7
    at Array.forEach (native)
    at Function._.each._.forEach (/home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/node_modules/underscore/underscore.js:78:11)
    at exports.connect (/home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:47:5)
    at /home/pi/hangouts-bots/node_modules/node-metainspector/node_modules/cheerio/lib/parse.js:64:7
    at Array.forEach (native)

If this is hard to solve, I will be glad to know if there is some workaround in order to, at least, keep my application running.

lorenzos avatar Jun 25 '15 16:06 lorenzos