use close instead of finish event for file copy completion (#3234)

Windows Defender updates timestamps on files immediately after a write stream closes. Moving utimes update to after the file has closed corrects this issue.

Author: sciolist

src/util/fs.js

@@ -523,7 +523,7 @@ export async function copyBulk(
         readStream.pipe(writeStream);
       });
 
-      writeStream.once('finish', function() {
+      writeStream.once('close', function() {
         fs.utimes(data.dest, data.atime, data.mtime, function(err) {
           if (err) {
             reject(err);