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

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

Github user jinmeiliao commented on the issue:

    https://github.com/apache/incubator-geode/pull/276
  
    A few things:
    1. In your RestSecurityPostProcessorTest, in the before method, you can get 
ahold of the region you created and pre populate the regions with a few keys 
and values, this way, you don't need to worry about putting in values in your 
tests.
    2. Looks like /{regions} end point also retrieves the values of the entire 
region, you need to post process the values there as well. 
    3. query end points definitely return region values as well.
    4. SecurityUtils.getSecurityService will definitely get the security 
service, but let's do it the spring way, I have a component called 
RestSecurityService, use that instead.


> 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)

Reply via email to