From 9a91ed5763ab591cab04801fc636287955c3fbc7 Mon Sep 17 00:00:00 2001 From: tkellner Date: Fri, 16 Dec 2011 10:16:44 +0000 Subject: Keep first RedirectURL. Note: this changes current behaviour of OnlineBKU! According to SL specification, the first RedirectURL should cause an immediate redirect, but until now, the last set RedirectURL took precedence (and was the only one honoured) git-svn-id: https://joinup.ec.europa.eu/svn/mocca/trunk@1005 8a26b1a7-26f0-462f-b9ef-d0e30c41f5a4 --- .../egiz/bku/binding/HTTPBindingProcessorImpl.java | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/binding/HTTPBindingProcessorImpl.java b/bkucommon/src/main/java/at/gv/egiz/bku/binding/HTTPBindingProcessorImpl.java index 9b0adb03..05290de2 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/binding/HTTPBindingProcessorImpl.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/binding/HTTPBindingProcessorImpl.java @@ -761,13 +761,20 @@ public class HTTPBindingProcessorImpl extends AbstractBindingProcessor implement } String charset = HttpUtil.getCharset(ct, true); assignXMLRequest(fp.getFormParameterValue(), charset); - } else { - FormParameterStore fps = new FormParameterStore(); - fps.init(fp); - //if (!fps.isEmpty()) { - log.debug("Setting form parameter: {}.", fps.getFormParameterName()); - formParameterMap.put(fps.getFormParameterName(), fps); - //} + } + else { + if (fp.getFormParameterName().equals(FixedFormParameters.REDIRECTURL) + && formParameterMap.containsKey(FixedFormParameters.REDIRECTURL)) { + log.info("Not updating previously set RedirectURL!"); + } + else { + FormParameterStore fps = new FormParameterStore(); + fps.init(fp); + //if (!fps.isEmpty()) { + log.debug("Setting form parameter: {}.", fps.getFormParameterName()); + formParameterMap.put(fps.getFormParameterName(), fps); + //} + } } } if (slCommand == null) { -- cgit v1.2.3