Merge branch 'develop' into pr/7

This commit is contained in:
Jaiver Pastor 2021-01-31 10:49:36 +01:00
commit 2bd56fc742
3 changed files with 23 additions and 4 deletions

View file

@ -1,6 +1,18 @@
# Changelog
## (unreleased)
## v3.0.4892 (2019-11-16)
### **Fixes**
- Fixed an issue where some users couldn't start Ombi
## v3.0.4887 (2019-11-13)
### **Fixes**
- Fixed issues with the RequestId Migration #3244 #3253 #3249
## v3.0.4880 (2019-11-13)
### **New Features**

View file

@ -96,7 +96,14 @@ namespace Ombi.Notifications
client.Authenticate(settings.Username, settings.Password);
}
_log.LogDebug("sending message to {0} \r\n from: {1}\r\n Are we authenticated: {2}", message.To, message.From, client.IsAuthenticated);
await client.SendAsync(message);
try
{
await client.SendAsync(message);
}
catch (MailKit.Net.Smtp.SmtpCommandException e) when (e.ErrorCode.Equals(MailKit.Net.Smtp.SmtpErrorCode.RecipientNotAccepted) && e.StatusCode.Equals(MailKit.Net.Smtp.SmtpStatusCode.MailboxUnavailable))
{
_log.LogError("Could not send email '{0}', address <{1}> does not exist.", message.Subject, model.To);
}
await client.DisconnectAsync(true);
}
}
@ -178,4 +185,4 @@ namespace Ombi.Notifications
}
}
}
}
}

View file

@ -985,7 +985,7 @@ namespace Ombi.Schedule.Jobs.Ombi
finalsb.Append("<br />");
}
var summary = info.summary;
var summary = info?.summary ?? string.Empty;
if (summary.Length > 280)
{
summary = summary.Remove(280);