diff options
Diffstat (limited to 'BKUOnline/src/main')
-rw-r--r-- | BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/SessionListener.java | 11 | ||||
-rw-r--r-- | BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionId.java | 38 | ||||
-rw-r--r-- | BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdFilter.java (renamed from BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/RequestIdFilter.java) | 28 | ||||
-rw-r--r-- | BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/TransactionIdResponseWrapper.java | 98 | ||||
-rw-r--r-- | BKUOnline/src/main/java/at/gv/egiz/bku/online/webapp/WebRequestHandler.java | 5 | ||||
-rw-r--r-- | BKUOnline/src/main/webapp/WEB-INF/web.xml | 2 |
6 files changed, 172 insertions, 10 deletions
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> |