delivery http does not clean temp file

[dazoot]

If you change the delivery to http in a plugin, zone-mta creates a temp file with the posted body in /tmp but fails to clean it, and you end up with huge /tmp junk.

            delivery.http = true;
            delivery.targetUrl = "https://somehost/someurl";

[andris9]

Well, this sounds strange, zone-mta should never write anything to disk 🤔

[dazoot]

I think it actually comes from: formidable.

this.uploadDir = opts.uploadDir || (os.tmpdir && os.tmpdir()) || os.tmpDir();

IncomingForm.prototype._uploadPath = function(filename) {
  var buf = crypto.randomBytes(16);
  var name = 'upload_' + buf.toString('hex');

  if (this.keepExtensions) {
    var ext = path.extname(filename);
    ext     = ext.replace(/(\.[a-z0-9]+).*/i, '$1');

    name += ext;
  }

  return path.join(this.uploadDir, name);
};

So it's not from delivery.http but from adding an API endpoint to zonemta which gets file as body.

app.addAPI("POST", "/save", (req, res, next) =>  {