Github user trixpan commented on the issue:

    https://github.com/apache/nifi/pull/827
  
    @joewitt, @olegz code is a great improvement to the previous code.
    
    I do think the the removal of the maximum message size and timeout checks 
should be addressed as soon as possible:
    
    I base my opinion solely on the fact their absence will cause issues to the 
clients sending emails/flows via this interface. 
    
    As the processor currently lacks the ability to provide the appropriate 
SMTP reply codes back to the connecting clients, both ListenSMTP and connecting 
MTAs will waste resource every time a a connection times out (I am not 100% 
sure how the size issue will unfold). 
    
    Postfix documentation describes this scenario:
    
    > smtp_data_done_timeout (default: 600s)
    > The Postfix SMTP client time limit for sending the SMTP ".", and for 
receiving the remote SMTP server response.
    > 
    > When no response is received within the deadline, a warning is logged 
that the mail may be delivered multiple times.
    
    Note the whooping 600s before postfix gives up and disconnect.
    
    Having said that, I am happy to defer this decision to the wider group.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to