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