Почему unzip работает не корректно при вызове из python с помощью subprocess.Popen?

Question

[enabl3]

Всем привет, столкнулся с такой проблемой.
Есть бот, которому отправляю ZIP архив, он его сохраняет у меня на компе, а после должен его распаковать в другую папку, а далее уже другие действия... Для этого есть bash скрипт, при выполнении которого из консоли, все это дело успешно работает (архив распаковывается без ошибок и все ОК).
Но через бота, данный скрипт почему то не работает, из python все это дело выглядит так:

proc = subprocess.Popen('test.sh', stdout=subprocess.PIPE)
                output = proc.stdout.read()
                result = output
                bot.send_message(message.chat.id, result)

Ошибка в логах следующая:

End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of (путь к архиву), period.

Answer 1

[Владимир]

зачем делать разархивирование внешним, если можно элегантно покодить ?

import zipfile
zip_ref = zipfile.ZipFile(path_to_zip_file, 'r')
zip_ref.extractall(directory_to_extract_to)
zip_ref.close()

Comments

[enabl3]

Дело в том, что это может быть либо файл, либо архив с файлом. По этому в баш скрипте выполнил проверку на то, архив это или нет, а далее уже что нужно делать для каждого из них.
Можно ли выполнить эту проверку на python?

[Владимир]

конечно же можно :)

[enabl3]

Владимир, [ERROR] - root - File is not a zip file

[Владимир]

потому что чекать надо

zipfile.is_zipfile(filename)

Returns True if filename is a valid ZIP file based on its magic number, otherwise returns False. filename may be a file or file-like object too.

Changed in version 3.1: Support for file and file-like objects.

[enabl3]

Владимир, тут не понял, я делал так:

if os.path.splitext(src)[1] == ".zip":
     zip_ref = zipfile.ZipFile(filename, 'r')
     zip_ref.extractall("path")
     zip_ref.close()
     bot.send_message(chat_id, "It's a ZIP file")
else:
     bot.send_message(chat_id, "WRONG FILE!!!")

Файл сохраняется, в ответ прилетает что это ZIP-файл, но сам архив не распаковывается.

[Владимир]

расширение файла не самая хорошая идея для детекта типа файла. я бы даже сказал совсем плохая
в остальном все работает, проверял на своих проектах, читайте документацию

[enabl3]

Владимир, используя zipfile.is_zipfile(filename), мне идет ответ что это якобы не ZIP, как такое может быть? И почему этот же архив распаковывается, unzip-ом через bash а из python нет.
ХЗ уже как побороть...

[Владимир]

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

[enabl3]

Получал файл из чата так

file_info = bot.get_file(message.document.file_id)
downloaded_file = bot.download_file(file_info.file_path)
src = 'path' + message.document.file_name;
with open(src, 'wb') as new_file:
     new_file.write(downloaded_file)

По нему и выдавало ошибку что это не архив. Решилось очень банально, в конец нужно было добавить закрытие файла, в моем случае это - new_file.close() После чего все работает как часы.

Answer 2

[marxxt]

Дело, как мне кажется, не в питоне

Возьмите какой-нибудь архив, вычислите хэш, отправьте боту, вычислите хэш принятого архива, сравните