fix: null check

hermanho · Sep 14, 2023 · 6ff20eb · 6ff20eb
1 parent d8ff521
commit 6ff20eb
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 5 deletions.
diff --git a/src/Postal.AspNetCore/EmailService.cs b/src/Postal.AspNetCore/EmailService.cs
@@ -104,7 +104,7 @@ public async Task<MailMessage> CreateMailMessageAsync(Email email)
             var rawEmailString = await emailViewRenderer.RenderAsync(email);
             emailParser = new EmailParser(emailViewRenderer);
             var mailMessage = await emailParser.ParseAsync(rawEmailString, email);
-            if (mailMessage.From == null || mailMessage.From.Address == null)
+            if ((mailMessage.From == null || mailMessage.From.Address == null) && this.options.FromAddress != null)
             {
                 mailMessage.From = new MailAddress(this.options.FromAddress);
             }

diff --git a/src/Postal.AspNetCore/TemplateServices/TemplateService.cs b/src/Postal.AspNetCore/TemplateServices/TemplateService.cs
@@ -60,8 +60,11 @@ DiagnosticListener diagnosticListener
             {
                 RequestServices = _serviceProvider,
             };
-            httpContext.Features.Set<IEndpointFeature>(new EndpointFeature(httpContextData.Endpoint));
-            httpContext.Features.Set<IRouteValuesFeature>(new RouteValuesFeature() { RouteValues = httpContextData.RouteValues });
+            if (httpContextData != null)
+            {
+                httpContext.Features.Set<IEndpointFeature>(new EndpointFeature(httpContextData.Endpoint));
+                httpContext.Features.Set<IRouteValuesFeature>(new RouteValuesFeature() { RouteValues = httpContextData.RouteValues });
+            }
 
             if (viewModel.RequestPath != null)
             {

diff --git a/src/Postal.Tests/EmailViewRenderTests.cs b/src/Postal.Tests/EmailViewRenderTests.cs
@@ -100,7 +100,7 @@ public FakeView(Func<ViewContext, string> templateString)
                 TemplateString = templateString;
             }
 
-            public string Path => throw new NotImplementedException();
+            public string Path => "FakePathUnitTest";
 
             public Func<ViewContext, string> TemplateString { get; private set; }
 

diff --git a/src/Postal.Tests/Postal.Tests.csproj b/src/Postal.Tests/Postal.Tests.csproj
@@ -31,4 +31,5 @@
     <None Update="postal.png">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
-  </ItemGroup></Project>
+  </ItemGroup>
+</Project>