Merge pull request #88 from billrob/dev

nblumhardt · web-flow · commit 0c65484c4d5d · 2019-03-13T14:57:06.000+10:00
Check for directory existence before attempting access.
diff --git a/src/Serilog.Sinks.File/Sinks/File/RollingFileSink.cs b/src/Serilog.Sinks.File/Sinks/File/RollingFileSink.cs
@@ -117,8 +117,11 @@ void OpenFile(DateTime now, int? minSequence = null)
             var existingFiles = Enumerable.Empty<string>();
             try
             {
-                existingFiles = Directory.GetFiles(_roller.LogFileDirectory, _roller.DirectorySearchPattern)
+                if (Directory.Exists(_roller.LogFileDirectory))
+                {
+                    existingFiles = Directory.GetFiles(_roller.LogFileDirectory, _roller.DirectorySearchPattern)
                                          .Select(Path.GetFileName);
+                }
             }
             catch (DirectoryNotFoundException) { }
 

Original file line number	Diff line number	Diff line change
`@@ -117,8 +117,11 @@ void OpenFile(DateTime now, int? minSequence = null)`
`117`	`117`	`var existingFiles = Enumerable.Empty<string>();`
`118`	`118`	`try`
`119`	`119`	`{`
`120`		`- existingFiles = Directory.GetFiles(_roller.LogFileDirectory, _roller.DirectorySearchPattern)`
	`120`	`+ if (Directory.Exists(_roller.LogFileDirectory))`
	`121`	`+ {`
	`122`	`+ existingFiles = Directory.GetFiles(_roller.LogFileDirectory, _roller.DirectorySearchPattern)`
`121`	`123`	`.Select(Path.GetFileName);`
	`124`	`+ }`
`122`	`125`	`}`
`123`	`126`	`catch (DirectoryNotFoundException) { }`
`124`	`127`