hanxianlong commented on a change in pull request #813:
URL: https://github.com/apache/james-project/pull/813#discussion_r777844536
##########
File path:
server/protocols/protocols-smtp/src/main/java/org/apache/james/smtpserver/SendMailHandler.java
##########
@@ -70,6 +70,8 @@ public HookResult onMessage(SMTPSession session, Mail mail) {
LOGGER.debug("sending mail");
try {
+ Message message = mail.getMessage();
+ message.setSubject(mail.getMessage().getSubject());
Review comment:
I don't thinks it IS necessary in my experience, because it looks like
the codes do nothing but just re-set the subject of the message...........
However, If I remove this two lines, the error code of subject issue will
not be solved. I don't know why, do you have any other suggestions?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
