felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fmesc...@apache.org
Subject svn commit: r981812 - /felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
Date Tue, 03 Aug 2010 10:32:47 GMT
Author: fmeschbe
Date: Tue Aug  3 10:32:47 2010
New Revision: 981812

URL: http://svn.apache.org/viewvc?rev=981812&view=rev
Log:
URL Decode the path info to make sure the value can properly be processed (looks like some
servlet containers do not decode this themselves)

Modified:
    felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java

Modified: felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java?rev=981812&r1=981811&r2=981812&view=diff
==============================================================================
--- felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
(original)
+++ felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/compendium/ConfigManager.java
Tue Aug  3 10:32:47 2010
@@ -97,7 +97,7 @@ public class ConfigManager extends Confi
         if ( pid == null )
         {
             String info = request.getPathInfo();
-            pid = info.substring( info.lastIndexOf( '/' ) + 1 );
+            pid = WebConsoleUtil.urlDecode( info.substring( info.lastIndexOf( '/' ) + 1 )
);
         }
 
         // the filter to select part of the configurations
@@ -403,7 +403,7 @@ public class ConfigManager extends Confi
                 }
                 catch (IllegalArgumentException t)
                 {
-                    // MetaTypeProvider.getObjectClassDefinition might throw illegal 
+                    // MetaTypeProvider.getObjectClassDefinition might throw illegal
                     // argument exception. So we must catch it here, otherwise the
                     // other configurations will not be shown
                     // See https://issues.apache.org/jira/browse/FELIX-2390



Mime
View raw message