Expressjs:上传图片时ENOENT

时间:2013-07-10 09:14:27

标签: javascript node.js file-upload express

我已经编写了一些用于上传头像的中间件,如下所示:

var gm = require('gm'),
    mkdirp = require('mkdirp'),
    fs = require('fs');

uploadAvatar = function(req, res, next) {
  var img, path, user;
  if (req.files.avatar_image) {
    user = req.user;
    img = req.files.avatar_image;
    path = __dirname + "/../../public/avatar/" + (user.name.parameterize()) + ".png";
    mkdirp.sync(__dirname + "/../../public/avatar/");
    fs.createReadStream(img.path).pipe(fs.createWriteStream(path));
    gm(path).resize(250, 250).autoOrient().quality(90).write(path, function(err) {
      if (err != null) {
        req.flash('error', err);
      } else {
        user.avatar = "/avatar/" + (user.name.parameterize()) + ".png";
        user.save(function(err) {
          if (err != null) {
            req.flash('error', err);
          }
          next();
        });
      }
    });
  } else {
    next();
  }
};

// Usage
app.post('/upload', ensureAuthenticated, uploadAvatar, handleUpload);

当我现在尝试上传图片时,node因非常有用的错误消息而崩溃:

events.js:72
    throw er; // Unhandled 'error' event
          ^
Error: ENOENT, open '/tmp/1126846a248af5c584770b07de649f9b.png'

我也试过在使用gm之前复制文件。我怀疑express在我甚至可以触摸之前删除文件,作为“安全”。

有人可以帮我吗?

编辑Full source

1 个答案:

答案 0 :(得分:3)

首先将临时文件复制到您的头像目录,然后确保您对头像以及临时目录和文件具有权限。

管道读取到写入流不是同步操作,并且在初始化读取句柄后会尝试直接管道,这可能会导致一些问题。
您需要添加事件并等到文件被复制:

var complete = function(err) {
  if (!err) {
    // process your gm
  } else {
    console.log(err);
  }
}

var read = fs.createReadStream(sourcePath);
read.on('error', function(err) {
  complete(err);
});

var write = fs.createWriteStream(targetPath);
write.on('error', function(err) {
  complete(err);
});
write.on('close', function() {
  complete();
});
read.pipe(write);