github/Ombi
1
0
Fork 0
mirror of https://github.com/Ombi-app/Ombi.git synced 2025-07-13 00:32:57 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Resolved #224 , Removed the 'SSL' option from the email notification settings. We will now use the correct secure socket options (SSL/TLS) for your email host.

Browse source
This commit is contained in:
tidusjar 2016-05-23 11:40:22 +01:00
parent d0e6a2a341
commit 8cce80654b
3 changed files with 4 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

1
PlexRequests.Core/SettingModels/EmailNotificationSettings.cs
View file

@ -36,6 +36,5 @@ namespace PlexRequests.Core.SettingModels
public bool Enabled { get; set; }
public bool EnableUserEmailNotifications { get; set; }
public string RecipientEmail { get; set; }
public bool Ssl { get; set; }
}
}

7
PlexRequests.Services/Notification/EmailMessageNotification.cs
View file

@ -25,9 +25,10 @@
// ************************************************************************/
#endregion
using System;
using System.Net;
using System.Net.Mail;
using System.Threading.Tasks;
using MailKit.Security;
using MimeKit;
using NLog;
@ -162,7 +163,7 @@ namespace PlexRequests.Services.Notification
{
using (var client = new SmtpClient())
{
client.Connect(settings.EmailHost, settings.EmailPort, settings.Ssl);
client.Connect(settings.EmailHost, settings.EmailPort, SecureSocketOptions.Auto); // Let MailKit figure out the correct SecureSocketOptions.
// Note: since we don't have an OAuth2 token, disable
// the XOAUTH2 authentication mechanism.

14
PlexRequests.UI/Views/Admin/EmailNotifications.cshtml
View file

@ -30,20 +30,6 @@
</div>
</div>
<div class="form-group">
<div class="checkbox">
@if (Model.Ssl)
{
<input type="checkbox" id="Ssl" name="Ssl" checked="checked"><label for="Ssl">SSL Enabled</label>
}
else
{
<input type="checkbox" id="Ssl" name="Ssl"><label for="Ssl">SSL Enabled</label>
}
</div>
</div>
<div class="form-group">
<div class="checkbox">