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

jinmeiliao 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 2b6e375  GEODE-2296: Internal message SIZE should not have 
authorization checks (#1169)
2b6e375 is described below

commit 2b6e375b5ab0ceff2e62f8b1d618b4852069e934
Author: jinmeiliao <[email protected]>
AuthorDate: Fri Dec 15 08:31:05 2017 -0800

    GEODE-2296: Internal message SIZE should not have authorization checks 
(#1169)
---
 .../org/apache/geode/internal/cache/tier/sockets/command/Size.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/Size.java
 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/Size.java
index 70dfe54..6122464 100644
--- 
a/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/Size.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/command/Size.java
@@ -31,8 +31,6 @@ import org.apache.geode.internal.i18n.LocalizedStrings;
 import org.apache.geode.internal.logging.log4j.LocalizedMessage;
 import org.apache.geode.internal.security.SecurityService;
 import org.apache.geode.security.GemFireSecurityException;
-import org.apache.geode.security.ResourcePermission.Operation;
-import org.apache.geode.security.ResourcePermission.Resource;
 
 public class Size extends BaseCommand {
 
@@ -92,7 +90,9 @@ public class Size extends BaseCommand {
 
     // Size the entry
     try {
-      securityService.authorize(Resource.DATA, Operation.READ, regionName);
+      // GEODE-4102
+      // securityService.authorize(Resource.DATA, Operation.READ, regionName);
+
       writeSizeResponse(region.size(), clientMessage, serverConnection);
     } catch (RegionDestroyedException rde) {
       writeException(clientMessage, rde, false, serverConnection);

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

Reply via email to