Add a little bit more structure to message header storage to generally avoid having to loop across headers to find the right one.

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@954414 13f79535-47bb-0310-9956-ffa450edef68

3 files changed, 7 insertions, 6 deletions

diff --git a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/RequestContextImpl.java b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/RequestContextImpl.java
index cdb510f7ec..1b2d79e4f1 100644
--- a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/RequestContextImpl.java
+++ b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/context/impl/RequestContextImpl.java
@@ -45,7 +45,7 @@ public class RequestContextImpl implements RequestContext {
     public Subject getSecuritySubject() {
         Subject subject = null;
         
-        for (Object header : ThreadMessageContext.getMessageContext().getHeaders()){
+        for (Object header : ThreadMessageContext.getMessageContext().getHeaders().values()){
             if (header instanceof Subject){
                 subject  = (Subject)header;
                 break;
diff --git a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/NonBlockingInterceptor.java b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/NonBlockingInterceptor.java
index 6435efe4d6..a31a126977 100644
--- a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/NonBlockingInterceptor.java
+++ b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/NonBlockingInterceptor.java
@@ -19,6 +19,7 @@
 package org.apache.tuscany.sca.core.invocation;
 
 import java.util.List;
+import java.util.Map;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -170,7 +171,7 @@ public class NonBlockingInterceptor implements Interceptor {
             throw new UnsupportedOperationException();
         }
         
-        public List<Object> getHeaders() {
+        public Map<String, Object> getHeaders() {
             return null;
         }        
         public <T> T getBindingContext() {
diff --git a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/impl/MessageImpl.java b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/impl/MessageImpl.java
index e66fb0567a..cc8cb48cc5 100644
--- a/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/impl/MessageImpl.java
+++ b/sca-java-2.x/trunk/modules/core/src/main/java/org/apache/tuscany/sca/core/invocation/impl/MessageImpl.java
@@ -18,8 +18,8 @@
  */
 package org.apache.tuscany.sca.core.invocation.impl;
 
-import java.util.ArrayList;
-import java.util.List;
+import java.util.HashMap;
+import java.util.Map;
 
 import org.apache.tuscany.sca.assembly.Endpoint;
 import org.apache.tuscany.sca.assembly.EndpointReference;
@@ -32,7 +32,7 @@ import org.apache.tuscany.sca.invocation.Message;
  * @version $Rev $Date$
  */
 public class MessageImpl implements Message { 
-    private List<Object> headers = new ArrayList<Object>();
+    private Map<String, Object> headers = new HashMap<String, Object>();
     private Object body;
     private Object messageID;
     private boolean isFault;
@@ -99,7 +99,7 @@ public class MessageImpl implements Message {
         this.operation = op;
     }
     
-    public List<Object> getHeaders() {
+    public Map<String, Object> getHeaders() {
         return headers;
     }