don't bother with async on the File.Delete

OrchardCMS · Nov 7, 2024 · c445821 · c445821
1 parent a582ee8
commit c445821
Showing 1 changed file with 1 addition and 10 deletions.
diff --git a/src/OrchardCore.Modules/OrchardCore.Media.Indexing.Pdf/Services/PdfMediaFileTextProvider.cs b/src/OrchardCore.Modules/OrchardCore.Media.Indexing.Pdf/Services/PdfMediaFileTextProvider.cs
@@ -16,7 +16,7 @@ public async Task<string> GetTextAsync(string path, Stream fileStream)
         {
             if (!fileStream.CanSeek)
             {
-                seekableStream = GetTemporaryFileStream();
+                seekableStream = new FileStream(Path.GetTempFileName(), FileMode.Create, FileAccess.Write, FileShare.None, 4096, FileOptions.DeleteOnClose | FileOptions.Asynchronous);
 
                 await fileStream.CopyToAsync(seekableStream);
 
@@ -38,16 +38,7 @@ public async Task<string> GetTextAsync(string path, Stream fileStream)
             if (seekableStream != null)
             {
                 await seekableStream.DisposeAsync();
-
-                await Task.Run(() => File.Delete(seekableStream.Name));
             }
         }
     }
-
-    private static FileStream GetTemporaryFileStream()
-    {
-        var tempFilePath = Path.Combine(Path.GetTempPath(), Path.GetTempFileName());
-
-        return new FileStream(tempFilePath, FileMode.Create, FileAccess.Write);
-    }
 }