[
https://issues.apache.org/jira/browse/GEODE-9980?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17512470#comment-17512470
]
ASF subversion and git services commented on GEODE-9980:
--------------------------------------------------------
Commit e6a1a403bfa20d054c1a8b4242ed2c632de0198f in geode's branch
refs/heads/support/1.14 from Kirk Lund
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=e6a1a40 ]
Revert "GEODE-9980: Improve error handling of serial filters (#7355)" (#7489)
This reverts commit 5d9e4b5b54b84c2de28b670fe6feb0b0c44443f7.
Revert "GEODE-9758: Add internal serial filter API (#7217)"
This reverts commit f0809e411d561226f736adca9d40cbbe27033e53.
Revert "GEODE-9879: Extract SystemProperty to geode-common (#7177)"
This reverts commit 2652bd65cd4192068de6c601f7ba92d120816593.
Revert "GEODE-9758: Move ClassUtils to geode-common (#7166)"
This reverts commit e821d6eff4541ae478a0eb67217bcb456abae97a.
Revert "GEODE-9758: Move SanctionedSerializables to filter package (#7165)"
This reverts commit 3f66ab05e5683edb4bcb43e52304d8e5f58ac707.
Revert "GEODE-9817: Enable customized source set paths for ClassAnalysisRule
(#7367)"
This reverts commit 8f3186cb74ccd2eb1ec8ba648c7a11b08d575962.
> Startup of Locator or Server should fail fast if
> geode.enableGlobalSerialFilter is enabled but fails configuration
> ------------------------------------------------------------------------------------------------------------------
>
> Key: GEODE-9980
> URL: https://issues.apache.org/jira/browse/GEODE-9980
> Project: Geode
> Issue Type: Bug
> Components: serialization
> Affects Versions: 1.15.0
> Reporter: Kirk Lund
> Assignee: Kirk Lund
> Priority: Major
> Labels: GeodeOperationAPI, blocks-1.15.0, pull-request-available
>
> The following error conditions need better handling which includes handling
> of all errors consistently and cause the startup of a Locator or Server to
> fail if it's unable to honor the setting of
> {{-Dgeode.enableGlobalSerialFilter=true}} for any reason. Currently, if
> {{-Dgeode.enableGlobalSerialFilter=true}} is specified but Geode is unable to
> create a global serial filter, then it will will log a warning and continue
> running. A user may easily miss that log statement and believe that the JVM
> is running with a properly configured serialization filter.
> 1) The user is trying to secure the JVM very thoroughly and accidentally
> specifies both {{-Djdk.serialFilter}} and
> {{-Dgeode.enableGlobalSerialFilter}}.
> 2) The user runs some non-Geode code in the same JVM that invokes
> {{ObjectInputFilter.Config.setFilter(...)}} directly.
> 3) The user is using a version of Java 8 prior to 8u121 (the release that
> first added {{sun.misc.ObjectInputFilter}}) and specifies
> {{-Dgeode.enableGlobalSerialFilter=true}}. Also, the same behavior occurs if
> they do NOT specify enabling that property.
> 4) {{LocatorLauncher}} or {{ServerLauncher}} is started in a JVM that has
> already created at least one {{ObjectInputStream}} which will cause
> {{ObjectInputFilter.Config.setFilter(...)}} to fail.
> 5) {{LocatorLauncher}} or {{ServerLauncher}} is started in a Java 8 JVM that
> is not based on OpenJDK (ie {{sun.misc.ObjectInputFilter}} does not exist).
> 6) {{LocatorLauncher}} or {{ServerLauncher}} is started in an unforeseen
> environment that causes invocation of
> {{ObjectInputFilter.Config.setFilter(...)}} via Java Reflection to throw
> {{IllegalAccessException}}.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
