diff options
| author | tkellner <tkellner@8a26b1a7-26f0-462f-b9ef-d0e30c41f5a4> | 2011-12-16 10:16:44 +0000 |
|---|---|---|
| committer | tkellner <tkellner@8a26b1a7-26f0-462f-b9ef-d0e30c41f5a4> | 2011-12-16 10:16:44 +0000 |
| commit | 9a91ed5763ab591cab04801fc636287955c3fbc7 (patch) | |
| tree | 1f9449efc996c3f3b924c0d707b5343896cfcfa7 | |
| parent | 671ab33373a24dbbaad5dd6224c5d2fcc70246bd (diff) | |
| download | mocca-9a91ed5763ab591cab04801fc636287955c3fbc7.tar.gz mocca-9a91ed5763ab591cab04801fc636287955c3fbc7.tar.bz2 mocca-9a91ed5763ab591cab04801fc636287955c3fbc7.zip | |
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
| -rw-r--r-- | bkucommon/src/main/java/at/gv/egiz/bku/binding/HTTPBindingProcessorImpl.java | 21 |
1 files 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) { |