Revert "Исправление уязвимости DotNetZip"

Author: EvilBeaver

Build.csproj

@@ -1,4 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
+    <Import Project="$(MSBuildExtensionsPath)\MSBuildCommunityTasks\MSBuild.Community.Tasks.Targets" />
+    
     <PropertyGroup>
         <ReleaseNumber Condition="'$(ReleaseNumber)' == ''">1.1.0</ReleaseNumber>
         <BuildNumber Condition="'$(BUILD_NUMBER)' != ''">$(BUILD_NUMBER)</BuildNumber>
@@ -28,7 +30,7 @@
         </ItemGroup>
 
         <Delete Files="@(TestsResults)" />
-        <RemoveDir Directories="$(ArtifactsRoot)" Condition="Exists($(ArtifactsRoot))" />
+        <DeleteTree Directories="$(ArtifactsRoot)" Condition="Exists($(ArtifactsRoot))" />
 
         <MSBuild Projects="$(Solution)" Targets="restore;Clean" Properties="ReleaseNumber=$(ReleaseNumber);Configuration=$(Configuration);Platform=%(PlatformItem.MSBuildName);TargetFrameworkId=$(TargetFrameworkId)"/>
 
@@ -42,7 +44,7 @@
             <TempContent Include="$(ArtifactsRoot)/tmp"/>
             <TempContent Include="$(MDDocFolder)"/>
         </ItemGroup>
-        <RemoveDir Directories="@(TempContent)"/>
+        <DeleteTree Directories="@(TempContent)"/>
     </Target>
 
     <Target Name="PrepareArtifactsRoot">
@@ -122,14 +124,14 @@
 
         <MakeDir Directories="$(LibFolder)" />
         <CallTarget Targets="InstallStdLib" Condition="'$(StandardLibraryPacks)' != ''"/>
-        <RemoveDir Directories="$(LibFolder)/opm" />
+        <DeleteTree Directories="$(LibFolder)/opm" />
 
         <Exec Command="&quot;$(ToolFolder)/oscript.exe&quot; &quot;$(MSBuildProjectDirectory)\install\unicode-zipper.os&quot; unpack &quot;$(MSBuildProjectDirectory)\install\opm.ospx&quot; &quot;$(LibFolder)\tmp&quot;"/>
         <Exec Command="&quot;$(ToolFolder)/oscript.exe&quot; &quot;$(MSBuildProjectDirectory)\install\unicode-zipper.os&quot; unpack &quot;$(LibFolder)\tmp\content.zip&quot; &quot;$(LibFolder)/opm&quot;"/>
 
         <Copy SourceFiles="$(MSBuildProjectDirectory)\install\package-loader.os" DestinationFolder="$(LibFolder)"/>
 
-        <RemoveDir Directories="$(LibFolder)\tmp"/>
+        <DeleteTree Directories="$(LibFolder)\tmp"/>
 
     </Target>
 

install/install.iss

@@ -62,7 +62,7 @@ Source: "{#ArtifactRoot}\{#Binaries}\ScriptEngine.dll"; DestDir: "{app}\bin"; Co
 Source: "{#ArtifactRoot}\{#Binaries}\OneScript.DebugProtocol.dll"; DestDir: "{app}\bin"; Components: main
 Source: "{#ArtifactRoot}\{#Binaries}\OneScript.DebugServices.dll"; DestDir: "{app}\bin"; Components: main
 Source: "{#ArtifactRoot}\{#Binaries}\OneScript.Language.dll"; DestDir: "{app}\bin"; Components: main
-Source: "{#ArtifactRoot}\{#Binaries}\ProDotNetZip.dll"; DestDir: "{app}\bin"; Components: main
+Source: "{#ArtifactRoot}\{#Binaries}\DotNetZip.dll"; DestDir: "{app}\bin"; Components: main
 Source: "{#ArtifactRoot}\{#Binaries}\Newtonsoft.Json.dll"; DestDir: "{app}\bin"; Components: main
 Source: "{#ArtifactRoot}\{#Binaries}\oscript.cfg"; DestDir: "{app}\bin"; Components: main; Flags: onlyifdoesntexist
 

src/Installer/BinaryContent.wxs

@@ -8,7 +8,7 @@
 
       <Component Id="HostedLib" Guid="A98FCBE4-04BB-44EA-809F-A375CE5B11D3">
         <File Source="$(var.OutputPathForBuild)bin\ScriptEngine.HostedScript.dll"></File>
-        <File Source="$(var.OutputPathForBuild)bin\ProDotNetZip.dll"></File>
+        <File Source="$(var.OutputPathForBuild)bin\DotNetZip.dll"></File>
         <File Source="$(var.OutputPathForBuild)bin\Newtonsoft.Json.dll"></File>
       </Component>
 

src/OneScriptDocumenter/AssemblyLoader.cs

@@ -58,7 +58,7 @@ public AssemblyLoader(string baseDirectory)
             _enumerationTypeAttribute = scriptEngineLib.GetType("ScriptEngine.EnumerationTypeAttribute", true);
             _enumValue = scriptEngineLib.GetType("ScriptEngine.EnumItemAttribute", true);
 
-            foreach (var name in new string[] { "ScriptEngine.HostedScript", "ProDotNetZip", "Newtonsoft.Json" })
+            foreach (var name in new string[] { "ScriptEngine.HostedScript", "DotNetZip", "Newtonsoft.Json" })
             {
                 var libFile = Path.Combine(_baseDirectory, name + ".dll");
                 if (File.Exists(libFile))

src/OneScriptDocumenter/packages.config

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="ProDotNetZip" version="1.20.0" targetFramework="net45" />
+  <package id="DotNetZip" version="1.11.0" targetFramework="net45" />
   <package id="MarkdownDeep.NET" version="1.5" targetFramework="net45" />
   <package id="Newtonsoft.Json" version="11.0.2" targetFramework="net452" />
 </packages>

src/ScriptEngine.HostedScript/Library/TextEncodingEnum.cs

@@ -81,10 +81,10 @@ public EnumerationValue System
 
         public EnumerationValue GetValue(Encoding encoding)
         {
-            if (encoding.Equals(Encoding.GetEncoding(866)) || encoding.CodePage == 866)
+            if (encoding.Equals(Encoding.GetEncoding(866)))
                 return Oem;
 
-            if (encoding.Equals(Encoding.GetEncoding(1251)) || encoding.CodePage == 1251)
+            if (encoding.Equals(Encoding.GetEncoding(1251)))
                 return Ansi;
 
             if (encoding.Equals(new UnicodeEncoding(false, true)))
@@ -93,7 +93,7 @@ public EnumerationValue GetValue(Encoding encoding)
             if (encoding.Equals(new UTF8Encoding(true)))
                 return Utf8;
 
-            if (encoding.Equals(new UTF8Encoding(false)) || encoding.CodePage == 65001)
+            if (encoding.Equals(new UTF8Encoding(false)))
                 return Utf8NoBOM;
 
             if (encoding.Equals(Encoding.Default))

src/ScriptEngine.HostedScript/ScriptEngine.HostedScript.csproj

@@ -7,7 +7,7 @@
     <Platforms>AnyCPU</Platforms>
   </PropertyGroup>
   <ItemGroup>
-    <PackageReference Include="ProDotNetZip" Version="1.20.0" />
+    <PackageReference Include="DotNetZip" Version="1.13.3" />
     <PackageReference Include="Newtonsoft.Json" Version="13.0.2" />
     <ProjectReference Include="..\ScriptEngine\ScriptEngine.csproj" />
   </ItemGroup>
@@ -34,7 +34,7 @@
     <Message Text="Copying StandaloneRunner resources" />
     <ItemGroup>
       <CopyToStandalone Include="$(TargetPath)" />
-      <CopyToStandalone Include="$(TargetDir)ProDotNetZip.dll" />
+      <CopyToStandalone Include="$(TargetDir)DotNetZip.dll" />
       <CopyToStandalone Include="$(TargetDir)Newtonsoft.Json.dll" />
     </ItemGroup>
     <Copy SourceFiles="@(CopyToStandalone)" DestinationFolder="$(MSBuildProjectDirectory)\..\StandaloneRunner\" />

src/StandaloneRunner/StandaloneRunner.csproj

@@ -26,7 +26,7 @@
   <ItemGroup>
     <EmbeddedResource Include="ScriptEngine.dll" />
     <EmbeddedResource Include="ScriptEngine.HostedScript.dll" />
-    <EmbeddedResource Include="ProDotNetZip.dll" />
+    <EmbeddedResource Include="DotNetZip.dll" />
     <EmbeddedResource Include="Newtonsoft.Json.dll" />
     <EmbeddedResource Include="OneScript.Language.dll" />
   </ItemGroup>

tests/native-api/AddInNative.vcxproj

@@ -22,6 +22,7 @@
     <ProjectGuid>{55890DF2-D13E-4C89-A01D-79CAD6726246}</ProjectGuid>
     <RootNamespace>testCPP</RootNamespace>
     <Keyword>Win32Proj</Keyword>
+    <WindowsTargetPlatformVersion>10.0.18362.0</WindowsTargetPlatformVersion>
   </PropertyGroup>
   <Import Project="$(VCTargetsPath)\Microsoft.Cpp.Default.props" />
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'" Label="Configuration">

tests/zip/test-zip.os tests/zip.ostests/zip/test-zip.os renamed to tests/zip.os

@@ -31,10 +31,6 @@
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьПарольРусскиеБуквыФайловАрхива");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьШифрованиеЗипАрхива");
 
-	//Отключен до исправления библиотеки 
-	// https://github.com/EvilBeaver/OneScript/issues/870
-	//ВсеТесты.Добавить("ТестДолжен_ПроверитьСимволыЮникодаВИменахФайлов");
-	
 	Возврат ВсеТесты;
 
 КонецФункции
@@ -105,8 +101,8 @@
 
 	Попытка
 		юТест.ПроверитьРавенство("", Чтение.Элементы[0].Путь);
-		юТест.ПроверитьРавенство("test-zip.os", Чтение.Элементы[0].Имя);
-		юТест.ПроверитьРавенство("test-zip", Чтение.Элементы[0].ИмяБезРасширения);
+		юТест.ПроверитьРавенство("zip.os", Чтение.Элементы[0].Имя);
+		юТест.ПроверитьРавенство("zip", Чтение.Элементы[0].ИмяБезРасширения);
 	Исключение	
 		Чтение.Закрыть();
 		ВызватьИсключение;
@@ -357,7 +353,7 @@
 
 	ИмяРусскогоФайла = "ОбщиеФункции.os";
 
-	ПутьРусскогоФайла = ОбъединитьПути(ФайлСкрипта.Путь, "../testlib", ИмяРусскогоФайла);
+	ПутьРусскогоФайла = ОбъединитьПути(ФайлСкрипта.Путь, "testlib", ИмяРусскогоФайла);
 
 	ИмяАрхива = СоздатьВременныйФайл("zip");
 	Архив = Новый ЗаписьZipФайла(ИмяАрхива);
@@ -706,22 +702,3 @@
 	Файл = Новый Файл(НеверныйПутьФайла);
 	юТест.ПроверитьЛожь(Файл.Существует(), СообщениеОшибки + НеверныйПутьФайла);
 КонецПроцедуры
-
-// https://github.com/EvilBeaver/OneScript/issues/870
-Процедура ТестДолжен_ПроверитьСимволыЮникодаВИменахФайлов() Экспорт
-    ФайлАрхива = ОбъединитьПути(ТекущийСценарий().Каталог, "fixtures", "testUTF8.zip");
-    
-    ВременныйКаталог = СоздатьВременныйФайл();
-    СоздатьКаталог(ВременныйКаталог);
-    
-    ФайлZip = Новый ЧтениеZipФайла(ФайлАрхива);
-    Попытка
-        ФайлZip.ИзвлечьВсе(ВременныйКаталог);
-    Исключение
-        ФайлZip.Закрыть();
-        ВызватьИсключение;
-    КонецПопытки;
-    
-    ФайлZip.Закрыть();
-    
-КонецПроцедуры