fix: Email.AttachFromFilename does not dispose stream

JohnCampionJr · JohnCampionJr · commit 87441ae430b6 · 2024-09-09T07:56:43.000-04:00
closes #23
diff --git a/src/FluentEmail.Core/Email.cs b/src/FluentEmail.Core/Email.cs
@@ -461,7 +461,7 @@ public IFluentEmail Attach(IEnumerable<Attachment> attachments)
 
         public IFluentEmail AttachFromFilename(string filename,  string contentType = null, string attachmentName = null)
         {
-            var stream = File.OpenRead(filename);
+            var stream = new MemoryStream(File.ReadAllBytes(filename));
             Attach(new Attachment
             {
                 Data = stream,

Original file line number	Diff line number	Diff line change
`@@ -461,7 +461,7 @@ public IFluentEmail Attach(IEnumerable<Attachment> attachments)`
`461`	`461`
`462`	`462`	`public IFluentEmail AttachFromFilename(string filename, string contentType = null, string attachmentName = null)`
`463`	`463`	`{`
`464`		`- var stream = File.OpenRead(filename);`
	`464`	`+ var stream = new MemoryStream(File.ReadAllBytes(filename));`
`465`	`465`	`Attach(new Attachment`
`466`	`466`	`{`
`467`	`467`	`Data = stream,`