[nhin-d] 2 new revisions pushed by marc...@mirthcorp.com on 2014-11-20 23:57 GMT

[nhi...@googlecode.com]

2 new revisions:

Revision: c295250aa432
Branch: default
Author: marcusm
Date: Thu Nov 20 23:16:27 2014 UTC
Log: Issue 249 - fix multiple recipient issues on thread data - make
sure w...
https://code.google.com/p/nhin-d/source/detail?r=c295250aa432

Revision: bbc5bf6b5499
Branch: default
Author: marcusm
Date: Thu Nov 20 23:40:57 2014 UTC
Log: Merge with abb446dfe7661e682afd68956c241219c996a2e0
https://code.google.com/p/nhin-d/source/detail?r=bbc5bf6b5499

==============================================================================
Revision: c295250aa432
Branch: default
Author: marcusm
Date: Thu Nov 20 23:16:27 2014 UTC
Log: Issue 249 - fix multiple recipient issues on thread data - make
sure we clean the thread data before processing a new request
https://code.google.com/p/nhin-d/source/detail?r=c295250aa432

Modified:
/java/xd-common/src/main/java/org/nhindirect/xd/soap/DirectSOAPHandler.java

=======================================
---
/java/xd-common/src/main/java/org/nhindirect/xd/soap/DirectSOAPHandler.java
Wed Sep 10 00:55:38 2014 UTC
+++
/java/xd-common/src/main/java/org/nhindirect/xd/soap/DirectSOAPHandler.java
Thu Nov 20 23:16:27 2014 UTC
@@ -217,10 +217,15 @@
else
{
LOGGER.info("Handling an inbound message");
-
+
+ // Issue 249 - before handling the inbound case, we should
clear
+ // out the old thread data if we don't this the To: (SMTP
recipients) will
+ // append from the previous thread data
+ SafeThreadData.clean(Thread.currentThread().getId());
+
SafeThreadData threadData =
SafeThreadData.GetThreadInstance(Thread.currentThread().getId());

-
+
SOAPMessage msg = ((SOAPMessageContext)
context).getMessage();

ServletRequest sr = (ServletRequest)
context.get(MessageContext.SERVLET_REQUEST);
@@ -298,6 +303,9 @@
}
else if
(StringUtils.contains(node.getNodeName(), "to"))
{
+ // XDR-MULTIPLE-RECIPIENT-ISSUE - this
is the part where old thread data
+ // gets into the To: and will cause
unwanted recipients
+ // (see above for the clear)
String recipient =
node.getTextContent();
if (threadData.getDirectTo() == null){
threadData.setDirectTo(recipient);

==============================================================================
Revision: bbc5bf6b5499
Branch: default
Author: marcusm
Date: Thu Nov 20 23:40:57 2014 UTC
Log: Merge with abb446dfe7661e682afd68956c241219c996a2e0
https://code.google.com/p/nhin-d/source/detail?r=bbc5bf6b5499