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 Mon, 07 Nov 2016 18:29:59 GMT

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

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

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

    https://github.com/apache/incubator-geode/pull/276#discussion_r86836201
  
    --- Diff: geode-core/src/main/java/org/apache/geode/security/PostProcessor.java ---
    @@ -44,6 +47,9 @@ default void init(Properties securityProps) {}
        */
       Object processRegionValue(Object principal, String regionName, Object key, Object value);
     
    +  Collection<Object> processQueryResult(Object principal, Query query, Collection<String>
regions,
    --- End diff --
    
    I don't think we want to add this method in the interface. For query,  essentially it's
still getting value from region. Customers should not be burdened with another method to implement
if they just want to manipulate the data. See how query result get post processed in gfsh
queries. 


> 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