SAK-50284 Kernel email Message-ID must contain angle brackets

kernel/kernel-impl/src/main/java/org/sakaiproject/email/impl/BasicEmailService.java

@@ -1343,7 +1343,9 @@ protected void sendMessageAndLog(InternetAddress[] to, long start, MimeMessage m
 					buf.append(" with ").append(parts.getCount() - 1).append(" attachments");
 				}
 			}
-			catch (IOException ioe) {}
+			catch (IOException ioe) {
+			    log.debug("could not read message contents, {}", ioe);
+			}
 
 			if (log.isDebugEnabled())
 			{
@@ -1608,9 +1610,9 @@ else if (canUseCharset(message, CharacterSet.WINDOWS_1252) && canUseCharset(getS
 		/**
 		 * Override MimeMessage Message-ID to use Sakai serverId instead of hostname
 		 * https://javaee.github.io/javamail/FAQ#msgid
-		 * @throws MessagingException
-		 */
-		protected void updateMessageID() throws MessagingException
+         */
+		@Override
+        protected void updateMessageID() throws MessagingException
 		{
 			StringBuilder s = new StringBuilder();
 			// Unique string is <hashcode>.<id>.<currentTime><suffix>
@@ -1621,10 +1623,11 @@ protected void updateMessageID() throws MessagingException
 					append(System.currentTimeMillis()).
 					append('@').
 					append(serverConfigurationService.getServerName());
-			setHeader("Message-ID", s.toString());
+			setHeader("Message-ID", "<" + s + ">");
 		}
 
-		protected void updateHeaders() throws MessagingException
+		@Override
+        protected void updateHeaders() throws MessagingException
 		{
 			super.updateHeaders();
 			if (m_id != null)
@@ -1672,17 +1675,12 @@ private void addEncodedHeader(String header, String name) throws MessagingExcept
 						}
 				  }
 			 } 
-			 catch (MessagingException e) 
-			 {
-				  log.error("Email.MyMessage: exception: " + e, e);
-				  addHeaderLine(header);
-			 } 
-			 catch (UnsupportedEncodingException e)
+			 catch (MessagingException | UnsupportedEncodingException e)
 			 {
-				  log.error("Email.MyMessage: exception: " + e, e);
+                 log.error("Email.MyMessage: exception: {}", e, e);
 				  addHeaderLine(header);
 			 }
-		} 
+        }
 	}
 
 	public String propName(String propNameTemplate)
@@ -1692,7 +1690,6 @@ public String propName(String propNameTemplate)
 
 	public String propName(String propNameTemplate, String protocol)
 	{
-		String formattedName = String.format(propNameTemplate, protocol);
-		return formattedName;
+        return String.format(propNameTemplate, protocol);
 	}
 }