Jimp: Invalid file signature in JIMP in nodejs

[akshayseth]

I am using JIMP to convert my image to greyscale and to reduce its quality.But for 2% of the cases it is corrupting my image and throwing an error in the console- "Error: Invalid file signature at Parser._parseSignature (C:UsersAkshayDesktopdarwinnode_modulespngjs libparser.js:50:18)" Below if the problematic code:
I am using JIMP to convert my image to greyscale and to reduce its quality.But for 2% of the cases it is corrupting my image and throwing an error in the console-
"Error: Invalid file signature
at Parser._parseSignature (C:UsersAkshayDesktopdarwinnode_modulespngjs
libparser.js:50:18)"
Below if the problematic code:

 var ext=path.extname(dest);
      if(ext!='.jpg'){
       dest=replaceExt(dest, '.jpg');
      }
      console.log(path.extname(dest));
      var file = fs.createWriteStream(dest);
      ////console.log(url)
      if(url.indexOf('https')!=-1){
        //console.log("https")
        var request = https.get(url, function(response) {
        response.pipe(file);
        file.on('finish', function() {
          Jimp.read(dest).then(function (lennaa) {
            lennaa.resize(256, 256)            // resize
              .quality(90)                 // set JPEG quality
              .greyscale()                 // set greyscale
              .write(dest); // save
          }).catch(function (err) {
            console.error(err);
          });
          file.close(cb);  // close() is async, call cb after close completes.
        });
      }).on('error', function(err) { // Handle errors
        fs.unlink(dest); // Delete the file async. (But we don't check the result)
        if (cb) cb(err.message);
      });
      }

[ashrafkm]

+1

[anton-bot]

Were you able to find a solution?

[akshayseth]

Not till now.

[anton-bot]

My solution was (shame!!) to add a half-second delay if the first read fails, and retry the read after that delay.

[k1ker]

@catcher-in-the-try can you tell me, how to do it?

[anton-bot]

@k1ker I added the code on StackOverflow: https://stackoverflow.com/questions/46404120/invalid-file-signature-in-jimp-in-nodejs

Copied here:

let image;
try {
  image = await Jimp.read(filepath);
} catch (error) {
  debug(`Error reading ${filepath}. But we'll try again!`);

  // Wait half second, try again. What an ugly hack. It might as well work.
  let temp = await new Promise(resolve => setTimeout(resolve, 600));
  try {
    image = await Jimp.read(filepath);
    debug('Success reading file on second attempt!');
  } catch (error2) {
    // If totally failed, at least exit gracefully:
    this.session.send('The bot is a little busy now. Please try again.');
    this.session.endDialog();
  }
}

The key part is the delay:

  let temp = await new Promise(resolve => setTimeout(resolve, 600));

[akshayseth]

Patch not a solution.This brings to the area which is not yet explored.Will start developing a new library to replace JIMP.

[Durss]

Did you try with "close" instead of "finish" ?
file.on('close' function() { Jimp.read(dest) [...] })

Works for me.

[k1ker]

Can you anwer, how after
>

image = await Jimp.read(filepath);

I can use function (err, lenna)? Now, it's like: Jimp.read(template,
function (err, lenna) {

2017-10-24 5:29 GMT+03:00 Anton Ivanov [e-mail]:

@k1ker https://github.com/k1ker I added the code on StackOverflow:
https://stackoverflow.com/questions/46404120/invalid-
file-signature-in-jimp-in-nodejs

Copied here:

let image;try {
image = await Jimp.read(filepath);
} catch (error) {
debug(Error reading ${filepath}. But we'll try again!);

// Wait half second, try again. What an ugly hack. It might as well work.
let temp = await new Promise(resolve => setTimeout(resolve, 600));
try {
image = await Jimp.read(filepath);
debug('Success reading file on second attempt!');
} catch (error2) {
// If totally failed, at least exit gracefully:
this.session.send('The bot is a little busy now. Please try again.');
this.session.endDialog();
}
}

The key part is the delay:

let temp = await new Promise(resolve => setTimeout(resolve, 600));

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/349#issuecomment-338853372,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Admfa8WuJhAfyCNS72e-2P-tXt52FO4Vks5svUuMgaJpZM4Pildc
.

[hipstersmoothie]

@akshayseth can you provide a minimal example repo? thanks!

[hipstersmoothie]

Since I have not seen any response on this I'm gonna close it cause I cannot reproduce the issue. @Durss seems to have a solution. Feel free to open another issue if it persists!