mirror of
https://github.com/NodeBB/NodeBB.git
synced 2025-11-01 19:46:01 +01:00
fixed image upload, delete temp file if there is error
This commit is contained in:
@@ -152,7 +152,7 @@ var user = require('./../user.js'),
|
|||||||
}
|
}
|
||||||
|
|
||||||
var filename = uid + '-profileimg' + extension;
|
var filename = uid + '-profileimg' + extension;
|
||||||
var uploadPath = path.join(__dirname, '../', global.nconf.get('upload_path'), filename);
|
var uploadPath = path.join(process.cwd(), global.nconf.get('upload_path'), filename);
|
||||||
|
|
||||||
winston.info('Attempting upload to: '+ uploadPath);
|
winston.info('Attempting upload to: '+ uploadPath);
|
||||||
|
|
||||||
@@ -179,10 +179,10 @@ var user = require('./../user.js'),
|
|||||||
|
|
||||||
res.json({ path: imageUrl });
|
res.json({ path: imageUrl });
|
||||||
});
|
});
|
||||||
|
|
||||||
});
|
});
|
||||||
|
|
||||||
os.on('error', function(err) {
|
os.on('error', function(err) {
|
||||||
|
fs.unlinkSync(tempPath);
|
||||||
winston.error('[%d] %s', Date.now(), + err);
|
winston.error('[%d] %s', Date.now(), + err);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -62,6 +62,7 @@ var express = require('express'),
|
|||||||
app.use(require('less-middleware')({ src: path.join(__dirname, '../', 'public'), prefix:nconf.get('relative_path') }));
|
app.use(require('less-middleware')({ src: path.join(__dirname, '../', 'public'), prefix:nconf.get('relative_path') }));
|
||||||
app.use(nconf.get('relative_path'), express.static(path.join(__dirname, '../', 'public')));
|
app.use(nconf.get('relative_path'), express.static(path.join(__dirname, '../', 'public')));
|
||||||
app.use(express.bodyParser()); // Puts POST vars in request.body
|
app.use(express.bodyParser()); // Puts POST vars in request.body
|
||||||
|
|
||||||
app.use(express.cookieParser()); // If you want to parse cookies (res.cookies)
|
app.use(express.cookieParser()); // If you want to parse cookies (res.cookies)
|
||||||
app.use(express.compress());
|
app.use(express.compress());
|
||||||
app.use(express.session({
|
app.use(express.session({
|
||||||
|
|||||||
Reference in New Issue
Block a user