Hi,
On Jun 20, 2013, at 5:39 PM, Jonathan Fuerth <jfuerth(a)redhat.com> wrote:
This PageRequest interface is really similar to something Johannes
Barop just contributed, but his was a private inner class of Navigation. The fact that
this has cropped up twice in the past two days indicates that something is missing from
the public API :)
But! Couldn't we just express the hasPermission method like this:
> PermissionStatus hasPermission(User user, Page page);
We could create the Page widget instance itself, write the @PageState fields to it, and
then ask the permission resolver if we should be allowed to show it. This may be more
convenient in many cases because of the type coercion that the framework does when writing
page state fields. Does this make sense?
Currently we don't have a Page class only a Page Annotation, do you mean Page as the
whole component? If so there is a lot more to serialise than just that state. Wouldn't
it be more optimal to introduce the Request object like Johannes also has?
Going even further, why not use the exact same PermissionResolver interface as
PicketLink? Then we can stick them in the shared package and reuse them between client and
server. Since they're created by the bean manager, there's tons of stuff we could
share between client and server simply by injecting the contextual information we need in
order to make the decision.
If we go for exactly the same api as picket link we should rethink what we have now.
Because now I made it to be more security framework independent, that is also why I make
this interface in the first place to make easy to understand and specific for errai. If we
go and use the picket link interface in this case we can remove a lot of code, but then it
also makes sense to remove the abstraction for security frameworks. Do you agree?
I'm not saying they have to be shareable in all cases, but in many cases it should be
possible. "Write it once!" is one of the Errai design principles--at least
according to the marketing segment of our conference talks. ;-)
-Jonathan
On 2013-06-20, at 5:48 AM, Erik Jan de Wit wrote:
> Hi,
>
> So I've worked some more on fine grained security and I've come up with the
following inspired by picket link:
>
> public interface RequestPermissionResolver {
>
> public enum PermissionStatus {
> ALLOW, DENY, NOT_APPLICABLE
> }
>
> /**
> * Tests if the currently authenticated user has permission to 'see' the
specified page request.
> *
> * @param user the user to validate the pageRequest for
> * @param pageRequest The pageRequest for which the permission is required
> * @return ALLOW if the current user has the permission DENY or NOT_APPLICABLE.
> */
> PermissionStatus hasPermission(User user, PageRequest pageRequest);
>
> The PageRequest contains the name of the page and the state. By implementing this
interface the user can create logic if he wants to show the page that is about to get
shown to the user or not.
>
> The only problem I have now is what should we do when the user decides not to show
the page? I can see 4 possibilities:
>
> 1. Create a message that is shown on the interface somewhere
> The problems with this are, what message to show should be translatable and where /
how to show it, must
> also be customisable. Could also be helpful as a general error message framework?
>
> 2. Navigate to other page
> We could navigate to an other page, with a role SecurityError or something like
that. This will mean the user
> has full control of what will be shown when security errors occur.
>
> 3. We redirect to the login page
> This is kinda strange because one is already logged in and if there is no message
this is not really helping
>
> 4. Throw an exception
> This is also not really helping as there is no way for the user to do something at
this point.
>
> What do you guys think?
>
> Cheers,
> Erik Jan
>
>
> _______________________________________________
> errai-dev mailing list
> errai-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/errai-dev
_______________________________________________
errai-dev mailing list
errai-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/errai-dev