You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wookie.apache.org by sc...@apache.org on 2010/02/26 11:31:48 UTC

svn commit: r916639 - /incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java

Author: scottbw
Date: Fri Feb 26 10:31:48 2010
New Revision: 916639

URL: http://svn.apache.org/viewvc?rev=916639&view=rev
Log:
Proxy now only decodes URLs where attempting to parse the URL fails; see WOOKIE-118 for details (and thanks to Raido and Sander for pointing this out)

Modified:
    incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java

Modified: incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java?rev=916639&r1=916638&r2=916639&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/proxy/ProxyServlet.java Fri Feb 26 10:31:48 2010
@@ -238,10 +238,10 @@
 				throw new MalformedURLException("Unable to obtain url from args");
 
 			try {
+				fNewUrl = new URL(endPointURL);
+			} catch (Exception ex) {
 				// try decoding the URL
 				fNewUrl = new URL(URLDecoder.decode(endPointURL, "UTF-8"));
-			} catch (Exception ex) {
-				fNewUrl = new URL(endPointURL);
 			}
 		}