[jira] Commented: (CLEREZZA-316) Update where a GraphNode accessible in ssp bases on a graph injected with service

["Hasan (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CLEREZZA-316?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12918496#action_12918496 ] 

Hasan commented on CLEREZZA-316:
--------------------------------

I think checkRead is superfluous.

+       /**
+        * Returns the wrapped LockableMGraph if the caller has all access rights,
+        * otherwise an AccessControlException is thrown.
+        *
+        * @return the wrapped LockableMGraph.
+        */
+       public LockableMGraph getUnsecuredMGraph() {
+               checkWrite();
+               checkRead();
+               return wrapped;
+       }
+
 }

> Update where a GraphNode accessible in ssp bases on a graph injected with service
> ---------------------------------------------------------------------------------
>
>                 Key: CLEREZZA-316
>                 URL: https://issues.apache.org/jira/browse/CLEREZZA-316
>             Project: Clerezza
>          Issue Type: Improvement
>            Reporter: Tsuyoshi Ito
>            Assignee: Manuel Innerhofer
>
>  Fix if this can easily be prevented
>  
> Possibly: have a service returning an unsecured graph, maybe add a method getUnsecuredMGrah to SecuredMGraph, this method would check permissions while the retuned mgarph would no longer need to. => to do in a dedicated issue

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.