Conversation
|
Due to the limitation of ACE in the region, @BenedekFarkas, could you please test the health check or send me test credentials privately if you don't mind I remembered that I suffered from the configuration at the time of implementing the Azure Email |
src/OrchardCore.Modules/OrchardCore.Email.Azure/HealthChecks/AzureEmailHealthCheck.cs
Outdated
Show resolved
Hide resolved
src/OrchardCore.Modules/OrchardCore.Email.Azure/HealthChecks/AzureEmailHealthCheck.cs
Outdated
Show resolved
Hide resolved
sebastienros
left a comment
sebastienros
left a comment
There was a problem hiding this comment.
Once comments are resolved
BenedekFarkas
left a comment
BenedekFarkas
left a comment
There was a problem hiding this comment.
Navigating to /health/live throws error for me:
InvalidOperationException: Unable to resolve service for type 'OrchardCore.Email.Azure.Models.AzureEmailOptions' while attempting to activate 'OrchardCore.Email.Azure.HealthChecks.AzureEmailHealthCheck'.
src/OrchardCore.Modules/OrchardCore.Email.Azure/Extensions/AzureEmailHealthCheckExtensions.cs
Outdated
Show resolved
Hide resolved
src/OrchardCore.Modules/OrchardCore.Email.Azure/HealthChecks/Startup.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Benedek Farkas <benedek.farkas@lombiq.com>
src/OrchardCore.Modules/OrchardCore.Email.Azure/HealthChecks/AzureEmailHealthCheck.cs
Outdated
Show resolved
Hide resolved
|
Still waiting for changes |
|
Changes have been made. I might need to test it myself. I will try to get the test credentials |
|
@BenedekFarkas seems the API call is not working. I just replaced it with a send call to an invalid email. Just confirm it's not charged. FYI in noticed from Metrics that a failing email is not counted, but I need your confirmation to merge this PR |
4 participants
No description provided.