can we stop changing the way multer setup please ? im banging my head

This used to work inside a route now it is no longer working

var handler = multer({

                dest: __base + 'uploads/images/',

                onFileUploadStart: function (file) {
                    // You now have access to req
                    console.dir(req);
                    console.log(file.fieldname + ' is starting ...')
                },

                onFileUploadComplete: function (file) {
                    console.log(file.fieldname + ' uploaded to  ' + file.path)
                    var asset = new Asset();
                    asset.path          = req.files.file ? req.files.file.path.replace('uploads','') : '';
                    asset.type          = req.params.type;
                    asset.base64_data   = req.body.base64_data ? req.body.base64_data : '';

                    asset.save(function(err, asset_data) {
                        if (err)
                            res.send(err);

                        res.json({ message: 'Asset created!', asset_data: asset_data });
                    });
                }

            });

            handler(req, res);

[gabeio]

TLDR; read the new readme for the new major version changes in multer (which are very minor) and learn the semver major version meaning.

The point behind having major version number changes is that it is not backwards compatible if you wish to use the older version we are not stopping you. But there are significant benefits to upgrading including that I by myself can not DoS your server by filling your hard drive in (your available hard drive space/your internet speed). As such these changes need a different API which you will need to learn, if you are not willing to learn new APIs when they change I can not help you. In essence you basically are trying to tell use to stop updating multer by telling us to not change anything.

[LinusU]

You were actually using DiskStorage all along 😃

multer({ dest: './photos/' })

is a shorthand for

multer({ storage: multer.diskStorage({ destination: './photos/' }) })

Ahh, I understand. It really refers to the filename parameter to the diskStorage function. Should probably clarify that.