Skip to content

Commit

Permalink
Do not assume foler name
Browse files Browse the repository at this point in the history
Fixes #1665
  • Loading branch information
mikeminutillo committed Jun 5, 2019
1 parent 273be1f commit 74e317d
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 14 deletions.
18 changes: 11 additions & 7 deletions src/ServiceControlInstaller.Engine/FileSystem/MonitoringZipInfo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ public void ValidateZip()
public bool TryReadMonitoringReleaseDate(out DateTime releaseDate)
{
releaseDate = DateTime.MinValue;
var tempFile = Path.Combine(Path.GetTempPath(), $@"ServiceControl.Monitoring\{Constants.MonitoringExe}");
try
{
using (var zip = ZipFile.Read(FilePath))
Expand All @@ -77,18 +76,23 @@ public bool TryReadMonitoringReleaseDate(out DateTime releaseDate)
return false;
}

entry.Extract(Path.GetTempPath(), ExtractExistingFileAction.OverwriteSilently);
return ReleaseDateReader.TryReadReleaseDateAttribute(tempFile, out releaseDate);
var tempPath = Path.GetTempPath();
var tempFile = Path.Combine(tempPath, entry.FileName);
try
{
entry.Extract(tempPath, ExtractExistingFileAction.OverwriteSilently);
return ReleaseDateReader.TryReadReleaseDateAttribute(tempFile, out releaseDate);
}
finally
{
File.Delete(tempFile);
}
}
}
catch
{
return false;
}
finally
{
File.Delete(tempFile);
}
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ public void ValidateZip()
public bool TryReadServiceControlReleaseDate(out DateTime releaseDate)
{
releaseDate = DateTime.MinValue;
var tempFile = Path.Combine(Path.GetTempPath(), $@"ServiceControl\{Constants.ServiceControlExe}");
try
{
using (var zip = ZipFile.Read(FilePath))
Expand All @@ -73,18 +72,23 @@ public bool TryReadServiceControlReleaseDate(out DateTime releaseDate)
return false;
}

entry.Extract(Path.GetTempPath(), ExtractExistingFileAction.OverwriteSilently);
return ReleaseDateReader.TryReadReleaseDateAttribute(tempFile, out releaseDate);
var tempPath = Path.GetTempPath();
var tempFile = Path.Combine(tempPath, entry.FileName);
try
{
entry.Extract(tempPath, ExtractExistingFileAction.OverwriteSilently);
return ReleaseDateReader.TryReadReleaseDateAttribute(tempFile, out releaseDate);
}
finally
{
File.Delete(tempFile);
}
}
}
catch
{
return false;
}
finally
{
File.Delete(tempFile);
}
}
}
}

0 comments on commit 74e317d

Please sign in to comment.