Ошибка метода ночной сборки "Новый ЧтениеZipФайла()"

[ViktorErmakov]

При попытке прочитать zip архив, выдает ошибку что в пути переданном на чтении содержатся недопустимые символы.

Воспроизведение ошибки

1. Запустить код:
   ZipФайлОбновления = "C:\1cv8.zip";
   ФайлZip = Новый ЧтениеZipФайла(ZipФайлОбновления);

2. Увидеть ошибку
   

Окружение

• ОС: Windows 7
• Версия OS: 1.1.0.325

Дополнительная информация
Приложить архив не получается, он более 10 метров, который пытается считать скрипт, но этот архив с сайта 1С обновление на последнюю версию УТ 10.3, и проблема не в нем точно.

[nixel2007]

Дополнительная информация из телеги - на стабильном движке работает, проблема в ночнике.

[EvilBeaver]

Принято

[EvilBeaver]

Запустил на чистых исходниках develop - ошибки нет. Код примера скопировал из этой задачи.
Просьба уточнить способ вопроизведения. У меня нет ошибки на вот этом коде в TestApp.

ZipФайлОбновления = "C:\1cv8.zip";
ФайлZip = Новый ЧтениеZipФайла(ZipФайлОбновления);         

[EvilBeaver]

Видимо дело в архиве. Можно выложить куда-то на файлообменник и прислать ссылку в личку (в телеграм или на ovsiankin.aa@gmail.com

[ViktorErmakov]

https://yadi.sk/d/iEtjT_N1p-sosA
Может конечно и в архиве, но очень странно все, на версии что я указал выше ошибка воспроизводилась, после удаления и установки стабильной версии, стало работать нормально.

[EvilBeaver]

Путь в архиве таки содержит недопустимые знаки:

При открытии архива штатными архиваторами видим, что там используется Юникодная кавычка «». Где-то в дебрях ICSharpZibLib она превращается в угловую скобку. Занятно, нечего сказать :)

[EvilBeaver]

Encoding.GetEncoding(866).GetString(new byte[]{60}) = "<"

Ну вот как-то так .NET конвертирует символы. Есть ли у кого-то идеи, почему именно так? @dmpas?

[EvilBeaver]

интересующее место в строке: индексы 34-36 по переходу с32-60-49САспект

32 - пробел
60 - символ '<'
49 - символ '1'

В архиве от 1С - "по переходу с «1САспект.."

[Mr-Rm]

Проблема в исходном архиве, точнее в архиваторе, которым он сделан.
Имена в формате UTF-8 хранятся в zip-архиве в ExtraField, а базовое имя - в кодировке 866.
Архиватор 1С переводит символы, отсутствующие в cp866 по своим правилам, в частности, « и » меняются на < и > соответственно. Из Руководство по переходу с «1САспект 7.7» на «1CУправление торговлей 8» получается Руководство по переходу с <1САспект 7.7> на <1CУправление торговлей 8>.
Видимо, ICSharpZibLib работает только с базовым именем, игнорируя имя в ExtraField.

[EvilBeaver]

@Mr-Rm ICSharpZibLib поддерживает флаг Юникода, но отрабатывает его, видимо не всегда. Я уже делал 1 PR и 2 issue в этот проект. Его явно забросили. Нашел актуализированную библиотеку DotNetZip, которую добрые люди портировали под Netstandard.

http://build.oscript.io/job/1Script/job/feature%252Fzip-restore/1/

Запустил сборку на новой (старой) либе. Тест @ViktorErmakov на ней прошел.

[EvilBeaver]

Тесты прошли. Кто может проверить на линуксе корректность полученного .deb пакета?

http://build.oscript.io/job/1Script/job/feature%252Fzip-restore/1/artifact/output/onescript-engine_1.1.0_all.deb

@nixel2007 @khorevaa @dmpas @pumbaEO ?

[Mr-Rm]

Ubuntu 18.04.1 x64: ФайлZip открывается, но ФайлZip.ИзвлечьВсе создает файл с именем Руководство по переходу с <1САспект 7.7>..., игнорируя имя в UTF-8.

[EvilBeaver]

@Mr-Rm а если на убунту поставить просто сам движок из приведенного выше deb - там пакет корректный? В нем русские файлы в папке lib - нормальные?

[Mr-Rm]

Именно из того пакета и поставил.
Посмотрел в lib. Нет, русские имена в неверной кодировке.

[Mr-Rm]

Просмотрел исходники и DotNetZip, и ICSharpZibLib. В них никак не обрабатывается Unicode Path Extra Field. Все флаги работы с кодовыми страницами и Юникодом действуют только на основное имя файла. Поэтому обе библиотеки с такими zip-файлами будут поступать согласно с

OneScript/src/ScriptEngine.HostedScript/Library/Zip/ZipReader.cs

Line 49 in 75fd033

// fuck non-russian encodings on non-ascii files