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
Created FedoraWebACUserSecurityContext #896
Conversation
return true; | ||
} else if ("http://www.w3.org/ns/auth/acl#Append".equals(roleName)) { | ||
return true; | ||
} else if ("http://www.w3.org/ns/auth/acl#Control".equals(roleName)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to define these constants. Should we move URIContants from the WebAC project to here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned above, we do not want to introduce "web-ac" concepts into the core codebase. We should figure out how to instead move this class into the fcrepo-module-auth-webac project.
Move FedoraWebACUserSecurityContext to WebAC project. Moved FedoraUserSecurityContext instantiation to fad
/** | ||
* Get an instance of FedoraUserSecurityContext that is compatible with the auth module. | ||
* @param userPrincipal | ||
* @return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added fedora-auth-common test-jar build for extension in webac project.
Is this ticket ready for review? |
Resolved with: 3b03cc1 |
No description provided.