You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rave.apache.org by "Anthony Carlucci (Commented) (JIRA)" <ji...@apache.org> on 2011/10/14 22:58:11 UTC

[jira] [Commented] (RAVE-298) Page manipulations are unchecked

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

Anthony Carlucci commented on RAVE-298:
---------------------------------------

Code checked in:
- implemented DefaultPagePermissionEvaluator
- protected one PageService method: getPage
- added handleAccessDeniedException to o.a.r.p.w.a.rest.PageApi
- fixed permissionEvaluator bean definition in applicationContext-security.xml to prevent duplicate bean creation

Still TODO:
1) finish securing the rest of the PageService methods with permission annotations
2) refactor the handleAccessDeniedException up into an AbstractRestApi class since it is common functionality that all RestApi controllers will want
                
> Page manipulations are unchecked
> --------------------------------
>
>                 Key: RAVE-298
>                 URL: https://issues.apache.org/jira/browse/RAVE-298
>             Project: Rave
>          Issue Type: Bug
>    Affects Versions: 0.4-INCUBATING
>            Reporter: Jasha Joachimsthal
>            Assignee: Anthony Carlucci
>            Priority: Critical
>             Fix For: 0.5-INCUBATING
>
>
> Currently it's possible to add/move/delete a widget on a page that does not belong to the logged in user by changing request parameters or the id in the url. Checks must be added to page and widget manipulations if the user that does the manipulations are performed by the owner.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira