[JENKINS-33999] Do no blacklist springs NestedRuntimeException

NestedRuntimeException needs to be propagated across remoting for things like remote authentication schemes.
jenkinsci · Apr 4, 2016 · d72ffb3 · d72ffb3
1 parent 7cb4138
commit d72ffb3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/main/java/hudson/remoting/ClassFilter.java b/src/main/java/hudson/remoting/ClassFilter.java
@@ -60,7 +60,7 @@ public final Class check(Class c) {
         ".*org[.]apache[.]xalan.*",
         "^org[.]codehaus[.]groovy[.]runtime[.].*",
         "^org[.]hibernate[.].*",
-        "^org[.]springframework[.].*",
+        "^org[.]springframework[.](?!core[.]NestedRuntimeException$).*",
         "^sun[.]rmi[.].*",
     };
 

diff --git a/src/test/java/hudson/remoting/DefaultClassFilterTest.java b/src/test/java/hudson/remoting/DefaultClassFilterTest.java
@@ -48,10 +48,11 @@ public class DefaultClassFilterTest {
     /** Some classes that should be matched by the default class filter */
     private static final List<String> defaultBadClasses = Arrays.asList("org.codehaus.groovy.runtime.Bob",
                                     "org.apache.commons.collections.functors.Wibble", "org.apache.xalan.Bogus",
-                                    "com.sun.org.apache.xalan.bogus");
+                                    "com.sun.org.apache.xalan.bogus", "org.springframework.core.SomeClass", 
+                                    "org.springframework.SomeClass");
     /** Some classes that should not be matched by the default class filter */
     private static final List<String> defaultOKClasses = Arrays.asList("java.lang.String", "java.lang.Object",
-                                    "java.util.ArrayList");
+                                    "java.util.ArrayList", "org.springframework.core.NestedRuntimeException");
 
     @Rule
     public TemporaryFolder folder = new TemporaryFolder();