search

ronysebastian / nhin-d

Automatically exported from code.google.com/p/nhin-d
0 stars 0 forks source link

DSN failure message is always sent if isReliableAndTimely=false #190

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Send a message without X-DIRECT-FINAL-DESTINATION-DELIVERY option

What is the expected output? What do you see instead?
successfulTransaction is not set to true if isReliableAndTimely=false in  
DirectXdMailet class. So that a DSN failure message is always sent to a sender.

What version of the product are you using? On what operating system?
xdmail-1.1.1

Original issue reported on code.google.com by teru.mor...@gmail.com on 17 Aug 2012 at 4:33

GoogleCodeExporter commented 9 years ago 
This is by design.  The RI takes the philosophy of being a "good neighbor HISP" 
and attempts to notify the sender if delivery is not successful.  This is not 
in violation of the applicability statement or the delivery notification 
implementation guide.  This is just good email practice if a message cannot be 
delivered regardless if the X-DIRECT-FINAL-DESTINATION-DELIVERY flag is set.

Original comment by gm2...@cerner.com on 17 Aug 2012 at 5:15

GoogleCodeExporter commented 9 years ago 
Issue is that DSN failure message is sent to the sender if delivery is 
successful.

Original comment by teru.mor...@gmail.com on 17 Aug 2012 at 5:34

GoogleCodeExporter commented 9 years ago 
My bad; I did not read your message correctly.  I don't have the ability to run 
this test with my setup and the library (I don't actually own this library) 
does not have tests for this.  I've asked a couple people to test, but I 
haven't heard back.  

I appreciate you taking this through a run.  I'll get the code updated and see 
if I can get some tests added to test all of the scenarios.

Original comment by gm2...@cerner.com on 17 Aug 2012 at 7:50

GoogleCodeExporter commented 9 years ago 
Code has been updated and pushed to the repository.  Please test and validate 
by replacing the xdmail-1.1.1.jar with the one below.

thx

https://oss.sonatype.org/content/repositories/snapshots/org/nhind/xdmail/1.1.2-S
NAPSHOT/xdmail-1.1.2-20120828.115050-1.jar

Original comment by gm2...@cerner.com on 28 Aug 2012 at 12:07

GoogleCodeExporter commented 9 years ago 
Issue is marked as fixed, but still needs external validation.

Original comment by gm2...@cerner.com on 24 Sep 2012 at 11:42

GoogleCodeExporter commented 9 years ago 
Validated.

Original comment by gm2...@cerner.com on 20 Nov 2012 at 3:34

GoogleCodeExporter commented 9 years ago 
Updating with the owner of the fix.

Original comment by gm2...@cerner.com on 20 Nov 2012 at 3:41