Как побороть непонятную ошибку fs.unlink в node.js?

Question

[sorokinfil]

Есть таблица в MySQL, в которой есть ссылки на картинки (оригиналы и превью). На ноде получаю все эти картинки, обхожу их в цикле и удаляю все. Если картинок больше одной пары (пара это оригинал и превью), нода выдаёт ошибку:
Error: EPERM: operation not permitted, unlink 'C:\путь_к_картинке\aa\bb\12345.jpg'
При этом сам файл 12345.jpg удаляется, а остальные (34772.jpg, 87202.jpg, ...) не удаляются, хотя должны. Решение для меня совершенно неочевидно, в интернете ничего дельного не нашёл, кроме даунгрейда npm до версии 5.3.0, который мне не помог.

Windows 7, командная строка запущена от имени администратора
Node.js 8.9.4
npm 5.3.0

Код цикла:

for (var i=0; i<result.length; i++)
				{
					if (result[i].img)
					{
						var path_image = path.resolve (CONFIG.STORAGE_DIR_PORTFOLIO, result[i].img);
						fs.open (path_image, 'r', (err, fd) =>
						{
							if (err)
							{
								/* Если файл не найден, то ничего не делаем */
								if (err.code === 'ENOENT')
									return;
								throw err;
							}
							
							/* Удаляем картинку */
							fs.unlink (path_image, function (err)
							{
								if (err)
									throw err;
							});
						});
					}
					if (result[i].image_preview)
					{
						var path_image_preview = path.resolve (CONFIG.STORAGE_DIR_PORTFOLIO, result[i].image_preview);
						fs.open (path_image_preview, 'r', (err, fd) =>
						{
							if (err)
							{
								/* Если файл не найден, то ничего не делаем */
								if (err.code === 'ENOENT')
									return;
								throw err;
							}
							
							/* Удаляем превью */
							fs.unlink (path_image_preview, function (err)
							{
								if (err)
									throw err;
							});
						});
					}
				}

Comments

[Сергей Сергей]

пишет "непозволительная операция". нужно права на файлы проверить как-то.

[sorokinfil]

Ну с правами точно должен быть порядок, ведь какие-то из картинок удаляются, а вот динамически заносятся все из них.
В параллельной таблице, аналогичная ситуация, а там все картинки проходят через изменение размера и пересохранение.
Мне кажется, что связано либо с ограничениями операционки, либо с ограничениями самого fs на количество удаляемых одновременно файлов, и я не имею понятия, как это можно пофиксить.

Answer 1

[RidgeA]

Есть подозрение, что проблема в том, что fs.open открывает файловый дескриптор, потому, скорее всего, файл и не удаляется корректно.

Comments

[rustler2000]

Поскольку это на венде - то скорее всего так оно и есть

[RidgeA]

венда, что с нее взять то....

[sorokinfil]

Если это и так, то можно ли как-то побороть? Я не нашёл в интернете таких ситуаций, что вообще странно, многие сидят на винде. Мб что-то в настройках системы изменить?

[rustler2000]

Филипп Сорокин, плохо гуглил - даже в репе ноды есть кейсы о проблемах на венде

Answer 2

[Владимир Скибин]

Не бойтесь использовать синхронные функции

result.forEach((item) => {
	return ((images) => {
		images.forEach((image) => {
			if (image) {
				let pathImg = path.resolve(CONFIG.STORAGE_DIR_PORTFOLIO, image);
				return fs.existsSync(pathImg) && fs.unlinkSync(pathImg);
			}
		})
	})([item.img, item.image_preview]);
});

Или Вам еще нужен результат удаления?

Comments

[sorokinfil]

Результат удаления не нужен, главное чтобы они просто удалялись, если ссылка поступает из базы и файл по ней существует. Ваш код почему-то не заработал, не удалилась ни одна, хотя в этот раз никаких ошибок не было. Наверно потому, что не указан обработчик. В любом случае спасибо, я думаю уже ограничиться синхронным удалением, чувствую, что с асинхронным придётся очень много мудрить.

[sorokinfil]

Поменял в своём коде unlink на unlinkSync, та же ошибка EPERM: operation not permitted, ни в какую не хочет ничего удалять.

[Владимир Скибин]

Филипп Сорокин, Покажите что у Вас приходит в result

[rustler2000]

Филипп Сорокин, потому что у тебя один и тотже файл пытается удалить ))))) к моменту когда колбэк от fs.open сработает - у тебя path_image[_preview] будет иной :D

[sorokinfil]

Владимир Скибин, вот вывел ссылки, которые возвращаются в цикле. Все разные, все картинки по путям точно существуют.


rustler2000, но ведь цикл работает синхронно, или я чего-то не понимаю? Переменная path_image объявляется и передаётся в unlink по значению, это не объект. Как она может вдруг поменяться? Почему, когда я меняю на unlinkSync, результат тот же самый? Там нет никакого коллбека.

[rustler2000]

Филипп Сорокин, это перед unlink? цикл то синхронный, но результат открытия с задержкой

[sorokinfil]

rustler2000, да, просто path.resolve возвращает строку, а не объект, а это простой тип и должен передаваться по значению. С моим пониманием асинхронности не вижу никаких противоречий. Самое смешное, что картинка, на которой он якобы спотыкается, удаляется, а остальные нет.

Переформулирую вопрос - как лучше всего сделать удаление файлов в цикле? Рекурсивной функцией с рекурсией по коллбеку?

[rustler2000]

Филипп Сорокин, да блин весь инет в этом - типа loop closure gotchas и пр.

Вот кури почему '2 - ' выводит только ccc :D

```
var strings = ['aaa', 'bbb', 'ccc']

function l(ss) {
setTimeout(function() {
console.log('3 - ' + ss)
}, 100);
}
for (i = 0; i < 3; ++i) {
var s = strings[i].trim()
console.log('1 - ' + s)
setTimeout(function() {
console.log('2 - ' + s)
}, 100)
l(s)
}
```

[sorokinfil]

rustler2000, я не js-разработчик по профилю, но мысль понятна, цикл отрабатывает прежде, чем срабатывает первый таймаут. Правильно ли я понимаю, что чтобы код работал, как надо, мне нужно обернуть path.resolve и fs.unlink в функцию? Уже ни в чём не уверен.

[sorokinfil]

rustler2000, обернул, больше ошибок не выдаёт. Дебаггинг на js то ещё удовольствие. Для меня было неочевидно, но сейчас всё кажется в порядке, спасибо!