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

Permission templates - declaring default ACLs for objects when created #55

Closed
2 tasks done
anarchivist opened this issue Oct 21, 2016 · 2 comments
Closed
2 tasks done

Comments

@anarchivist
Copy link
Member

anarchivist commented Oct 21, 2016

Tagging @jcoyne. We need to document our needs around permission templates, which are the default ACLs for an object when it is created.

  • Should permission templates be stored in Fedora? (@jcoyne says no; I'm not sure I agree).
  • Does acl:defaultForNew make sense here? What is the expectation/relation to what actually gets persisted to Fedora? (see here for more info on default perms)
@anarchivist
Copy link
Member Author

See Slack discussion onward.

@anarchivist
Copy link
Member Author

Decisions from 10/24 call (following sprint planning):

Should permission templates be stored in Fedora? (@jcoyne says no; I'm not sure I agree).

No, they should not go in Fedora (for now) -- it's faster to do this from the database, and @jcoyne suggested that we view the permission templates as ephemeral objects. However, this should be flagged by someone (@jcoyne? @mjgiarlo?) for discussion either in a Hydra tech call or with the Architecture Working Group.

Does acl:defaultForNew make sense here? What is the expectation/relation to what actually gets persisted to Fedora? (see here for more info on default perms)

No, it doesn't make sense to use acl:defaultForNew. Based on the above answer, and the assumption that we're doing copy on create (see AWG notes), we might want to mint a new predicate if we want to reference the permissions template in some way, or if it ultimately gets persisted in Fedora. (I'll document it as a MAY, with some notes that point to this issue.)

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

No branches or pull requests

1 participant