Maxxx873 commented on PR #2460:
URL: https://github.com/apache/james-project/pull/2460#issuecomment-2531564636

   > IMO we need a way to test this:
   > 
   > GIVEN From REQUIRETLS WHEN remoteDelivery speaks to a server that do NOT 
allow startTLS THEN it fails
   > 
   > Don't we have a way to tweak MockSMTP into allowing testing this.
   > 
   > And yes and VERY good work, by the way.
   
   Thanks so much for the review and remarks! Unfortunately, I don't know how 
to implement it correctly yet. I tried setting the MockSMTP behavior, but 
that's not what's needed as far as I understand. It is possible to make a tweak 
to MockSMTP to respond not only to commands, but also to parameters passed. Is 
it necessary? Further, it is not very clear how and at what point I can check 
the startTLS allowance for remoteDelivery.  Also a question regarding Bounce 
and DSNBounce maillets. The REQUIRETLS email attribute will be saved as it is, 
what do I need to add additionally? I apologize for the large scope of 
questions.


-- 
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]

Reply via email to