This is an automated email from the ASF dual-hosted git repository.

bschuchardt pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 9ff9211  GEODE-3923 Provide whitelist/blacklist capability for java 
serialization
9ff9211 is described below

commit 9ff92114d185b8c367d1746726f3ca96a58c508b
Author: Bruce Schuchardt <[email protected]>
AuthorDate: Thu Nov 30 16:05:47 2017 -0800

    GEODE-3923 Provide whitelist/blacklist capability for java serialization
    
    Fixing a few more minor review comments - removing commented out code
---
 .../org/apache/geode/internal/ObjectInputStreamFilterWrapper.java   | 6 ------
 .../internal/InternalDataSerializerSerializationWhitelistTest.java  | 2 +-
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/ObjectInputStreamFilterWrapper.java
 
b/geode-core/src/main/java/org/apache/geode/internal/ObjectInputStreamFilterWrapper.java
index 3552ac4..0e5bf83 100644
--- 
a/geode-core/src/main/java/org/apache/geode/internal/ObjectInputStreamFilterWrapper.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/ObjectInputStreamFilterWrapper.java
@@ -61,10 +61,6 @@ public class ObjectInputStreamFilterWrapper implements 
InputStreamFilter {
     }
 
     logger.info("setting a serialization filter containing {}", 
serializationFilterSpec);
-    // logger.info("sanctioned serializables are");
-    // for (String cls: sanctionedClasses) {
-    // logger.info(" " + cls);
-    // }
 
     final ObjectInputFilter userFilter =
         ObjectInputFilter.Config.createFilter(serializationFilterSpec);
@@ -89,8 +85,6 @@ public class ObjectInputStreamFilterWrapper implements 
InputStreamFilter {
       }
     };
 
-    // global filter - if we enable this it will affect all ObjectInputStreams
-    // ObjectInputFilter.Config.setSerialFilter(serializationFilter);
   }
 
   @Override
diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerSerializationWhitelistTest.java
 
b/geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerSerializationWhitelistTest.java
index 73690d9..74740a6 100644
--- 
a/geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerSerializationWhitelistTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerSerializationWhitelistTest.java
@@ -96,7 +96,7 @@ public class InternalDataSerializerSerializationWhitelistTest 
{
   }
 
   @Test(expected = java.io.InvalidClassException.class)
-  public void whitelistedWithNonlMatchingFilterCannotSerialize() throws 
Exception {
+  public void whitelistedWithNonMatchingFilterCannotSerialize() throws 
Exception {
     trySerializingWithFilter("RabidMonkeyTurnip");
   }
 

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to