You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@labs.apache.org by si...@apache.org on 2010/02/05 13:17:31 UTC

svn commit: r906913 - /labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java

Author: simoneg
Date: Fri Feb  5 12:17:31 2010
New Revision: 906913

URL: http://svn.apache.org/viewvc?rev=906913&view=rev
Log:
Missed on spot where tomcat/jetty incompatibilities may cause problems

Modified:
    labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java

Modified: labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java
URL: http://svn.apache.org/viewvc/labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java?rev=906913&r1=906912&r2=906913&view=diff
==============================================================================
--- labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java (original)
+++ labs/magma/trunk/foundation-website/src/main/java/org/apache/magma/website/utils/LinkHelper.java Fri Feb  5 12:17:31 2010
@@ -237,6 +237,10 @@
 				String converted = conv.to(param);
 				try {
 					converted = URLEncoder.encode(converted, "UTF-8");
+					// Jetty and Tomcat behaves differently in regard to "+" in path info.
+					// Jetty does not parse it while Tomcat converts it to space.
+					// Better not to have "+" at all.
+					converted = converted.replaceAll("\\+", "%20");
 				} catch (UnsupportedEncodingException e) {
 					// should never happen for UTF-8
 				}



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@labs.apache.org
For additional commands, e-mail: commits-help@labs.apache.org