Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SEC-1166: Provide strategy interface for AclImpl isGranted() method. #1414

Closed
spring-projects-issues opened this issue May 26, 2009 · 4 comments

Comments

@spring-projects-issues
Copy link

@spring-projects-issues spring-projects-issues commented May 26, 2009

Luke Taylor (Migrated from SEC-1166) said:

This will allow more flexibility in the way the permission bitmasks are evaluated. The existing auditLogger should be moved to the default implementation, as this is the only place logging occurs in AclImpl.

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Feb 20, 2010

Luke Taylor said:

Done. See "source" tab for information.

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Jul 11, 2011

Bojan Tomic said:

Does this mean providing a bitwise comparing strategy is a safe thing to do?

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Nov 25, 2015

Peter Rietzler said:

It would be nice if DefaultPermissionGrantingStrategy would contain a template method in order to decide if a permission is contained in the given mask - see
https://gist.github.com/oliverfernandez/36846fcdc03696a7b829
which is a copy with a new method containsPermission. If DefaultPermissionGrantingStrategy would adopt this with a protected version of containsPermission it would be easier to add stuff like bit-wise permission checking etc.

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Feb 6, 2016

This issue supersedes #740

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant