geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GEODE-1993) value returned through /region/key rest service needs to be post processed
Date Tue, 08 Nov 2016 00:28:58 GMT

    [ https://issues.apache.org/jira/browse/GEODE-1993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15645944#comment-15645944
] 

ASF GitHub Bot commented on GEODE-1993:
---------------------------------------

Github user kjduling commented on a diff in the pull request:

    https://github.com/apache/incubator-geode/pull/276#discussion_r86900814
  
    --- Diff: geode-core/src/main/java/org/apache/geode/internal/security/IntegratedSecurityService.java
---
    @@ -409,6 +435,34 @@ public Object postProcess(Object principal, String regionPath, Object
key, Objec
         return newValue;
       }
     
    +  private Object getPrincipal(Object principal) {
    +    if (principal == null) {
    +      Subject subject = getSubject();
    +      if (subject == null)
    +        return null;
    +      principal = subject.getPrincipal();
    +    }
    +    return principal;
    +  }
    +
    +  @Override
    +  public Collection<Object> postProcess(Query query, Collection<String> regionNames,
    +      Collection<Object> results) {
    +    return postProcess(null, query, regionNames, results);
    +  }
    +
    +  @Override
    +  public Collection<Object> postProcess(Object principal, Query query,
    --- End diff --
    
    done


> value returned through /region/key rest service needs to be post processed
> --------------------------------------------------------------------------
>
>                 Key: GEODE-1993
>                 URL: https://issues.apache.org/jira/browse/GEODE-1993
>             Project: Geode
>          Issue Type: New Feature
>          Components: management
>            Reporter: Jinmei Liao
>            Assignee: Kevin Duling
>             Fix For: 1.1.0-incubating
>
>
> The new rest security did not use post processor before returning the value back to the
client.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message