richardgirges / express-fileupload

Simple express file upload middleware that wraps around busboy

MIT License

1.52k stars 261 forks source link

tmp file isn't deleted if failed to .mv #307

Closed malletgaetan closed 2 years ago

malletgaetan commented 2 years ago

problem

if using tmp files for upload, in the case the file.mv fails, tmp file isn't removed#305

version

1.3.1

fix

in the case this is in fact not wanted, i'll give it a shot for pr

richardgirges commented 2 years ago

This sounds like intentional behavior to me. It should be up to the user to delete any auto-generated temp folders in the event that the .mv method fails.