Resolved issue [#MOCCA-745] (Two simultaneous requests within the same session cause failure in both requests).

git-svn-id: https://joinup.ec.europa.eu/svn/mocca/trunk@790 8a26b1a7-26f0-462f-b9ef-d0e30c41f5a4

14 files changed, 439 insertions, 42 deletions

diff --git a/BKUApplet/src/main/java/at/gv/egiz/bku/online/applet/BKUApplet.java b/BKUApplet/src/main/java/at/gv/egiz/bku/online/applet/BKUApplet.java
index 2876c56e..f6ceacd0 100644
--- a/BKUApplet/src/main/java/at/gv/egiz/bku/online/applet/BKUApplet.java
+++ b/BKUApplet/src/main/java/at/gv/egiz/bku/online/applet/BKUApplet.java
@@ -187,7 +187,13 @@ public class BKUApplet extends JApplet {
       log.warn("Cannot load applet background image. {}", ex.getMessage());
     }
 
-    helpListener = new HelpListener(getParameter(HELP_URL), getLocale());
+    try {
+      URL helpUrl = getURLParameter(HELP_URL);
+      helpListener = new HelpListener(helpUrl, getLocale());
+      log.debug("Setting help URL: {}", helpUrl);
+    } catch (MalformedURLException e) {
+      log.warn("Cannot set help URL. {}", e.getMessage());
+    }
 
     SwitchFocusListener switchFocusListener = new SwitchFocusListener(
             getAppletContext(), "focusToBrowser");
diff --git a/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/HelpListener.java b/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/HelpListener.java
index 2f6f1738..a0ab2d77 100644
--- a/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/HelpListener.java
+++ b/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/HelpListener.java
@@ -20,6 +20,8 @@ import java.awt.event.KeyEvent;
 import java.awt.event.KeyListener;
 import java.awt.event.MouseEvent;
 import java.awt.event.MouseListener;
+import java.net.MalformedURLException;
+import java.net.URL;
 import java.util.Locale;
 import java.util.ResourceBundle;
 import org.slf4j.Logger;
@@ -30,14 +32,16 @@ public class HelpListener implements MouseListener, KeyListener, HelpURLProvider
   public static final String MESSAGE_BUNDLE = "at/gv/egiz/bku/gui/Messages";
 
   private final Logger log = LoggerFactory.getLogger(HelpListener.class);
-  private String helpURL;
+
+  private URL baseURL;
+  
   protected String helpTopic;
   protected ResourceBundle messageBundle;
 
-  public HelpListener(String baseHelpURL, Locale locale) {
-
-    helpURL = baseHelpURL;
+  public HelpListener(URL baseURL, Locale locale) {
 
+    this.baseURL = baseURL;
+    
     if (locale != null) {
       log.trace("Check for support of requested help locale {}.", locale);
       messageBundle = ResourceBundle.getBundle(MESSAGE_BUNDLE, locale);
@@ -45,14 +49,18 @@ public class HelpListener implements MouseListener, KeyListener, HelpURLProvider
       messageBundle = ResourceBundle.getBundle(MESSAGE_BUNDLE);
     }
 
-    if ("".equals(messageBundle.getLocale().getLanguage())) {
-      log.trace("Using help locale 'default'.");
-    } else {
-      log.trace("Using help locale '{}'.", messageBundle.getLocale());
-      helpURL += messageBundle.getLocale().getLanguage() + '/';
-    }
-    
-    log.debug("Setting help context to {}.", helpURL);
+    String language = messageBundle.getLocale().getLanguage();
+    log.trace("Using help language '{}'.", language);
+    if (!language.isEmpty()) {
+      try {
+        baseURL = new URL(baseURL, language + "/");
+      } catch (MalformedURLException e) {
+        log.info("Failed to build baseURL using language {}. " +
+        		"Using default language.", language, e);
+      }
+    } 
+    log.debug("Setting help context to {}.", baseURL);
+
   }
 
   @Override
@@ -63,15 +71,15 @@ public class HelpListener implements MouseListener, KeyListener, HelpURLProvider
 
   @Override
   public synchronized String getHelpURL() {
-    if (helpTopic == null) {
-      log.debug("No help topic set, return index.");
-      return helpURL + "index.html";
+    try {
+      URL helpURL = new URL(baseURL, (helpTopic != null) ? "index.html" : helpTopic + ".html");
+      log.debug("Return help url: {}.", helpURL);
+      return helpURL.toString();
+    } catch (MalformedURLException e) {
+      log.info("Failed to build helpURL. Returning base URL: {}.", baseURL, e);
+      return baseURL.toString();
     }
-    StringBuilder url = new StringBuilder(helpURL);
-    url.append(helpTopic);
-    url.append(".html");
-    log.debug("Return help url: {}.", url.toString());
-    return url.toString();
+    
   }
 
   /**
diff --git a/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/ViewerHelpListener.java b/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/ViewerHelpListener.java
index 3d01c404..3c35e7b6 100644
--- a/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/ViewerHelpListener.java
+++ b/BKUCommonGUI/src/main/java/at/gv/egiz/bku/gui/ViewerHelpListener.java
@@ -33,12 +33,12 @@ public class ViewerHelpListener extends HelpListener {
 
   protected AppletContext appletCtx;
 
-  public ViewerHelpListener(String helpURL, Locale locale) {
-    super(helpURL, locale);
+  public ViewerHelpListener(URL baseURL, Locale locale) {
+    super(baseURL, locale);
   }
 
-  public ViewerHelpListener(AppletContext ctx, String helpURL, Locale locale) {
-    super(helpURL, locale);
+  public ViewerHelpListener(AppletContext ctx, URL baseURL, Locale locale) {
+    super(baseURL, locale);
     this.appletCtx = ctx;
   }
 
diff --git a/BKUCommonGUI/src/test/java/at/gv/egiz/bku/gui/SecureViewerDialogTest.java b/BKUCommonGUI/src/test/java/at/gv/egiz/bku/gui/SecureViewerDialogTest.java
index b927aecd..dde71390 100644
--- a/BKUCommonGUI/src/test/java/at/gv/egiz/bku/gui/SecureViewerDialogTest.java
+++ b/BKUCommonGUI/src/test/java/at/gv/egiz/bku/gui/SecureViewerDialogTest.java
@@ -12,6 +12,7 @@ import java.io.FileInputStream;
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.UnsupportedEncodingException;
+import java.net.URL;
 import java.util.ResourceBundle;
 import org.junit.After;
 import org.junit.AfterClass;
@@ -32,8 +33,9 @@ public class SecureViewerDialogTest {
 
   @BeforeClass
   public static void setUpClass() throws Exception {
+    URL baseURL = new URL("../help");
     messages = ResourceBundle.getBundle("at/gv/egiz/bku/gui/Messages");
-    secureViewer = new SecureViewerDialog(null, messages,null, null, new DummyFontLoader(), new HelpListener("../help", messages.getLocale()), 1f);
+    secureViewer = new SecureViewerDialog(null, messages,null, null, new DummyFontLoader(), new HelpListener(baseURL, messages.getLocale()), 1f);
   }
 
   @AfterClass
diff --git a/BKULocal/src/main/java/at/gv/egiz/bku/local/gui/LocalHelpListener.java b/BKULocal/src/main/java/at/gv/egiz/bku/local/gui/LocalHelpListener.java
index ced47d6c..8730a611 100644
--- a/BKULocal/src/main/java/at/gv/egiz/bku/local/gui/LocalHelpListener.java
+++ b/BKULocal/src/main/java/at/gv/egiz/bku/local/gui/LocalHelpListener.java
@@ -21,6 +21,7 @@ import java.awt.Desktop;
 import java.awt.event.KeyEvent;
 import java.awt.event.MouseEvent;
 import java.net.URI;
+import java.net.URL;
 import java.util.Locale;
 
 import org.slf4j.Logger;
@@ -37,8 +38,8 @@ public class LocalHelpListener extends ViewerHelpListener {
   
   protected Desktop desktop;
 
-  public LocalHelpListener(String helpURL, Locale locale) {
-    super(helpURL, locale);
+  public LocalHelpListener(URL baseURL, Locale locale) {
+    super(baseURL, locale);
     if (Desktop.isDesktopSupported() && Desktop.getDesktop().isSupported(Desktop.Action.BROWSE)) {
       desktop = Desktop.getDesktop();
     } else {
diff --git a/BKULocal/src/main/java/at/gv/egiz/bku/local/stal/LocalSTALFactory.java b/BKULocal/src/main/java/at/gv/egiz/bku/local/stal/LocalSTALFactory.java
index 2c986d84..cc0d8fc2 100644
--- a/BKULocal/src/main/java/at/gv/egiz/bku/local/stal/LocalSTALFactory.java
+++ b/BKULocal/src/main/java/at/gv/egiz/bku/local/stal/LocalSTALFactory.java
@@ -19,6 +19,8 @@ package at.gv.egiz.bku.local.stal;
 import at.gv.egiz.bku.viewer.ResourceFontLoader;
 import java.awt.Dimension;
 import java.awt.Toolkit;
+import java.net.MalformedURLException;
+import java.net.URL;
 import java.util.Locale;
 
 
@@ -46,7 +48,7 @@ public class LocalSTALFactory implements STALFactory {
 
   private final Logger log = LoggerFactory.getLogger(LocalSTALFactory.class);
   protected static final Dimension PREFERRED_SIZE = new Dimension(318, 200);
-  protected String helpURL;
+  protected URL helpURL;
   protected Locale locale;
   
   protected Configuration configuration;
@@ -108,9 +110,10 @@ public class LocalSTALFactory implements STALFactory {
   /**
    * spring injects helpURL
    * @param helpURL
+   * @throws MalformedURLException if helpURL is not a valid URL
    */
-  public void setHelpURL(String helpURL) {
-    this.helpURL = helpURL;
+  public void setHelpURL(String helpURL) throws MalformedURLException {
+    this.helpURL = new URL(helpURL);
   }
 
   /**
diff --git a/BKULocal/src/main/java/at/gv/egiz/bku/local/webapp/PINManagementServlet.java b/BKULocal/src/main/java/at/gv/egiz/bku/local/webapp/PINManagementServlet.java
index ab41634e..1d079786 100644
--- a/BKULocal/src/main/java/at/gv/egiz/bku/local/webapp/PINManagementServlet.java
+++ b/BKULocal/src/main/java/at/gv/egiz/bku/local/webapp/PINManagementServlet.java
@@ -22,6 +22,7 @@ import at.gv.egiz.stal.STAL;
 import at.gv.egiz.stal.STALResponse;
 import at.gv.egiz.stal.ext.PINManagementRequest;
 import java.io.IOException;
+import java.net.MalformedURLException;
 import java.net.URL;
 import java.util.Collections;
 import java.util.List;
@@ -46,7 +47,11 @@ public class PINManagementServlet extends HttpServlet {
 
   public PINManagementServlet() {
     stalFactory = new LocalSTALFactory();
-    stalFactory.setHelpURL("http://localhost:3495/help/");
+    try {
+      stalFactory.setHelpURL("http://localhost:3495/help/");
+    } catch (MalformedURLException e) {
+      log.info("Failed to set help URL.", e);
+    }
   }
 
   /**
diff --git a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/SessionListener.java b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/SessionListener.java
index e7f05862..58e9c861 100644
--- a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/SessionListener.java
+++ b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/SessionListener.java
@@ -23,6 +23,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import at.gv.egiz.bku.binding.BindingProcessorManager;
+import at.gv.egiz.bku.binding.Id;
 import at.gv.egiz.bku.binding.IdFactory;
 
 /**
@@ -36,12 +37,20 @@ public class SessionListener implements HttpSessionListener {
   @Override
   public void sessionCreated(HttpSessionEvent event) {
     log.info("Session {} created.", event.getSession().getId());
+    event.getSession().setAttribute(TransactionId.TRANSACTION_INDEX, new TransactionId());
   }
 
   @Override
   public void sessionDestroyed(HttpSessionEvent event) {
     BindingProcessorManager manager = (BindingProcessorManager) event.getSession().getServletContext().getAttribute("bindingProcessorManager");
-    manager.removeBindingProcessor(IdFactory.getInstance().createId(event.getSession().getId()));
+    TransactionId tidx = (TransactionId) event.getSession().getAttribute(TransactionId.TRANSACTION_INDEX);
+    if (tidx != null) {
+      IdFactory idFactory = IdFactory.getInstance();
+      for (int i = 0; i <= tidx.get(); i++) {
+        Id id = idFactory.createId(event.getSession().getId() + "-" + i);
+        manager.removeBindingProcessor(id);
+      }
+    }
     log.info("Session {} destroyed.", event.getSession().getId());
   }
 
diff --git a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionId.java b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionId.java
new file mode 100644
index 00000000..329db1bb
--- /dev/null
+++ b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionId.java
@@ -0,0 +1,38 @@
+/*
+* Copyright 2009 Federal Chancellery Austria and
+* Graz University of Technology
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*     http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
+
+package at.gv.egiz.bku.online.webapp;
+
+import java.io.Serializable;
+
+public class TransactionId implements Serializable {
+
+  private static final long serialVersionUID = 1L;
+
+  public static final String TRANSACTION_INDEX = "tidx";
+  
+  private int index = 0;
+  
+  public synchronized int next() {
+    return ++index;
+  }
+  
+  public synchronized int get() {
+    return index;
+  }
+  
+}
diff --git a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/RequestIdFilter.java b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdFilter.java
index 210f482f..3ae41707 100644
--- a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/RequestIdFilter.java
+++ b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdFilter.java
@@ -9,6 +9,7 @@ import javax.servlet.ServletException;
 import javax.servlet.ServletRequest;
 import javax.servlet.ServletResponse;
 import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpSession;
 
 import org.slf4j.MDC;
@@ -19,7 +20,7 @@ import at.gv.egiz.bku.binding.IdFactory;
 /**
  * Servlet Filter implementation class BindingProcessorFilter
  */
-public class RequestIdFilter implements Filter {
+public class TransactionIdFilter implements Filter {
   
   /**
    * @see Filter#destroy()
@@ -35,9 +36,28 @@ public class RequestIdFilter implements Filter {
 
     if (request instanceof HttpServletRequest) {
       HttpSession session = ((HttpServletRequest) request).getSession();
-      Id id = IdFactory.getInstance().createId(session.getId());
+      
+      String tidx = null;
+      // We expect the transaction index parameter to appear in GET requests only
+      if ("GET".equals(((HttpServletRequest) request).getMethod())) {
+        tidx = request.getParameter("tidx");
+      }
+
+      if (tidx == null) {
+        TransactionId transactionIndex = (TransactionId) session
+            .getAttribute(TransactionId.TRANSACTION_INDEX);
+        if (transactionIndex != null) {
+          tidx = Integer.toString(transactionIndex.next());
+        } else {
+          tidx = "0";
+        }
+      }
+      
+      Id id = IdFactory.getInstance().createId(session.getId() + "-" + tidx);
       MDC.put("id", id.toString());
       request.setAttribute("id", id);
+      
+      response = new TransactionIdResponseWrapper((HttpServletResponse) response, session.getId(), tidx);
     }
 
     // pass the request along the filter chain
@@ -54,5 +74,5 @@ public class RequestIdFilter implements Filter {
    */
   public void init(FilterConfig fConfig) throws ServletException {
   }
-
-}
+  
+} 
diff --git a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapper.java b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapper.java
new file mode 100644
index 00000000..c7bbf85c
--- /dev/null
+++ b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapper.java
@@ -0,0 +1,98 @@
+/*
+* Copyright 2009 Federal Chancellery Austria and
+* Graz University of Technology
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*     http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
+
+/**
+ * 
+ */
+package at.gv.egiz.bku.online.webapp;
+
+import javax.servlet.http.HttpServletResponse;
+import javax.servlet.http.HttpServletResponseWrapper;
+
+public class TransactionIdResponseWrapper extends HttpServletResponseWrapper {
+  
+  private String sessionId;
+  
+  private String tidx;
+
+  public TransactionIdResponseWrapper(HttpServletResponse response, String sessionId, String tidx) {
+    super(response);
+    this.sessionId = sessionId;
+    this.tidx = tidx;
+  }
+
+  /* (non-Javadoc)
+   * @see javax.servlet.http.HttpServletResponseWrapper#encodeRedirectURL(java.lang.String)
+   */
+  @Override
+  public String encodeRedirectURL(String url) {
+    // ensure jsessionid and tidx parameters
+    String encodedUrl = super.encodeRedirectUrl(url);
+    int i = encodedUrl.indexOf('?');
+    StringBuilder u = new StringBuilder();
+    if (i > 0) {
+      u.append(encodedUrl.substring(0, i));
+    } else {
+      u.append(encodedUrl);
+    }
+    if (!encodedUrl.contains(";jsessionid=")) {
+      u.append(";jsessionid=");
+      u.append(sessionId);
+    }
+    if (i < 0) {
+      u.append('?');
+    } else if (i < encodedUrl.length() - 1) {
+      u.append(encodedUrl.substring(i));
+      u.append('&');
+    }
+    u.append("tidx=");
+    u.append(tidx);
+    
+    return u.toString();
+  }
+
+  /* (non-Javadoc)
+   * @see javax.servlet.http.HttpServletResponseWrapper#encodeURL(java.lang.String)
+   */
+  @Override
+  public String encodeURL(String url) {
+    // ensure tidx parameter
+    String encodedUrl = super.encodeUrl(url);
+    if (url.endsWith("?wsdl")) {
+      // don't add parameters to ?wsdl URLs
+      return encodedUrl;
+    }
+    int i = encodedUrl.indexOf('?');
+    StringBuilder u = new StringBuilder();
+    if (i > 0) {
+      u.append(encodedUrl.substring(0, i));
+    } else {
+      u.append(encodedUrl);
+    }
+    if (i < 0) {
+      u.append('?');
+    } else if (i < encodedUrl.length() - 1) {
+      u.append(encodedUrl.substring(i));
+      u.append('&');
+    }
+    u.append("tidx=");
+    u.append(tidx);
+    
+    return u.toString();
+  }
+  
+}
\ No newline at end of file
diff --git a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/WebRequestHandler.java b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/WebRequestHandler.java
index 08b50699..0f432a10 100644
--- a/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/WebRequestHandler.java
+++ b/BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/WebRequestHandler.java
@@ -121,10 +121,7 @@ public class WebRequestHandler extends HttpServlet {
     bindingProcessorManager.process(id, bindingProcessor);
   
     log.debug("Sending redirect to user interface.");
-    // TODO: appending the jsessionid here breaks the separation of request
-    // handling and request state tracking done in the servlet filter, but 
-    // the servlet filter does not allow us to modify the redirect
-    resp.sendRedirect("ui;jsessionid=" + id.toString());
+    resp.sendRedirect(resp.encodeRedirectURL("ui"));
       
   }
 
diff --git a/BKUOnline/src/main/webapp/WEB-INF/web.xml b/BKUOnline/src/main/webapp/WEB-INF/web.xml
index 3a318844..bcce9b34 100644
--- a/BKUOnline/src/main/webapp/WEB-INF/web.xml
+++ b/BKUOnline/src/main/webapp/WEB-INF/web.xml
@@ -153,7 +153,7 @@
 	 -->
 	<filter>
 		<filter-name>RequestIdFilter</filter-name>
-		<filter-class>at.gv.egiz.bku.online.webapp.RequestIdFilter</filter-class>
+		<filter-class>at.gv.egiz.bku.online.webapp.TransactionIdFilter</filter-class>
 	</filter>
 	<filter-mapping>
 		<filter-name>RequestIdFilter</filter-name>
diff --git a/BKUOnline/src/test/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapperTest.java b/BKUOnline/src/test/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapperTest.java
new file mode 100644
index 00000000..b1c2ea53
--- /dev/null
+++ b/BKUOnline/src/test/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapperTest.java
@@ -0,0 +1,210 @@
+/*
+* Copyright 2009 Federal Chancellery Austria and
+* Graz University of Technology
+*
+* Licensed under the Apache License, Version 2.0 (the "License");
+* you may not use this file except in compliance with the License.
+* You may obtain a copy of the License at
+*
+*     http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing, software
+* distributed under the License is distributed on an "AS IS" BASIS,
+* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+* See the License for the specific language governing permissions and
+* limitations under the License.
+*/
+
+package at.gv.egiz.bku.online.webapp;
+
+import static org.junit.Assert.*;
+
+import java.io.IOException;
+import java.io.PrintWriter;
+import java.net.MalformedURLException;
+import java.net.URL;
+import java.util.Locale;
+
+import javax.servlet.ServletOutputStream;
+import javax.servlet.http.Cookie;
+import javax.servlet.http.HttpServletResponse;
+
+import org.junit.Test;
+
+
+public class TransactionIdResponseWrapperTest {
+
+  @Test
+  public void testEncodeRedirectURLfString() {
+    
+    String sessionId = "nQeu67vFMC+OTCXI67woBlpYgCxn";
+    String tidx = "2";
+    
+    TransactionIdResponseWrapper wrapper = new TransactionIdResponseWrapper(new ServletResponse(), sessionId, tidx);
+    
+    assertEquals(";jsessionid=nQeu67vFMC+OTCXI67woBlpYgCxn?tidx=2", wrapper.encodeRedirectURL(""));
+    assertEquals("ui;jsessionid=nQeu67vFMC+OTCXI67woBlpYgCxn?tidx=2", wrapper.encodeRedirectURL("ui"));
+    assertEquals("/mocca/ui;jsessionid=nQeu67vFMC+OTCXI67woBlpYgCxn?param1=test&tidx=2", wrapper.encodeRedirectURL("/mocca/ui?param1=test"));
+    
+  }
+
+  @Test
+  public void testEncodeURLString() {
+
+    String sessionId = "nQeu67vFMC+OTCXI67woBlpYgCxn";
+    String tidx = "2";
+    
+    TransactionIdResponseWrapper wrapper = new TransactionIdResponseWrapper(new ServletResponse(), sessionId, tidx);
+    
+    assertEquals("?tidx=2", wrapper.encodeURL(""));
+    assertEquals("ui?tidx=2", wrapper.encodeURL("ui"));
+    assertEquals("/mocca/ui?param1=test&tidx=2", wrapper.encodeURL("/mocca/ui?param1=test"));
+    assertEquals("stal?wsdl", wrapper.encodeUrl("stal?wsdl"));
+
+  }
+
+  public class ServletResponse implements HttpServletResponse {
+
+    @Override
+    public void addCookie(Cookie cookie) {
+    }
+
+    @Override
+    public void addDateHeader(String name, long date) {
+    }
+
+    @Override
+    public void addHeader(String name, String value) {
+    }
+
+    @Override
+    public void addIntHeader(String name, int value) {
+    }
+
+    @Override
+    public boolean containsHeader(String name) {
+      return false;
+    }
+
+    @Override
+    public String encodeRedirectURL(String url) {
+      return url;
+    }
+
+    @Override
+    public String encodeRedirectUrl(String url) {
+      return url;
+    }
+
+    @Override
+    public String encodeURL(String url) {
+      return url;
+    }
+
+    @Override
+    public String encodeUrl(String url) {
+      return url;
+    }
+
+    @Override
+    public void sendError(int sc) throws IOException {
+    }
+
+    @Override
+    public void sendError(int sc, String msg) throws IOException {
+    }
+
+    @Override
+    public void sendRedirect(String location) throws IOException {
+    }
+
+    @Override
+    public void setDateHeader(String name, long date) {
+    }
+
+    @Override
+    public void setHeader(String name, String value) {
+    }
+
+    @Override
+    public void setIntHeader(String name, int value) {
+    }
+
+    @Override
+    public void setStatus(int sc) {
+    }
+
+    @Override
+    public void setStatus(int sc, String sm) {
+    }
+
+    @Override
+    public void flushBuffer() throws IOException {
+    }
+
+    @Override
+    public int getBufferSize() {
+      return 0;
+    }
+
+    @Override
+    public String getCharacterEncoding() {
+      return null;
+    }
+
+    @Override
+    public String getContentType() {
+      return null;
+    }
+
+    @Override
+    public Locale getLocale() {
+      return null;
+    }
+
+    @Override
+    public ServletOutputStream getOutputStream() throws IOException {
+      return null;
+    }
+
+    @Override
+    public PrintWriter getWriter() throws IOException {
+      return null;
+    }
+
+    @Override
+    public boolean isCommitted() {
+      return false;
+    }
+
+    @Override
+    public void reset() {
+    }
+
+    @Override
+    public void resetBuffer() {
+    }
+
+    @Override
+    public void setBufferSize(int size) {
+    }
+
+    @Override
+    public void setCharacterEncoding(String charset) {
+    }
+
+    @Override
+    public void setContentLength(int len) {
+    }
+
+    @Override
+    public void setContentType(String type) {
+    }
+
+    @Override
+    public void setLocale(Locale loc) {
+    }
+    
+  }
+  
+}