Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

[DRP] Add DRP action name to Fidesops Policy Model #414

Closed
adriaaaa opened this issue Apr 26, 2022 · 2 comments · Fixed by #453
Closed

[DRP] Add DRP action name to Fidesops Policy Model #414

adriaaaa opened this issue Apr 26, 2022 · 2 comments · Fixed by #453
Assignees
Labels
enhancement New feature or request

Comments

@adriaaaa
Copy link

adriaaaa commented Apr 26, 2022

Requirements

  1. MUST have the ability to specify a “DRP action” when creating or editing a policy
  1. MUST require that a given DRP action is only used once (this ensures a 1:1 match from DRP actions to Fidesops policies)
  2. MUST validate the action against the list of valid actions defined in the DRP specification
  3. SHOULD NOT require that DRP actions are defined; it is valid to have multiple Fidesops policies defined but none mapped to a DRP action
  4. SHOULD NOT ascribe any specific behaviour to this DRP action, it is purely to translate between standards-based DRP actions and the user-defined policies in Fidesops

Implementation Notes

  • includes ingesting - the whole flow to associating an action with a policy
  • SQLAlchemy will allow us to specify that the DRP action field as unique.
  • validating the action can be db-level validation
  • see full requirements here
@adriaaaa adriaaaa added the enhancement New feature or request label Apr 26, 2022
@eastandwestwind
Copy link
Contributor

@adriaaaa one bit of clarification- by "focus", do you mean we should only support access and deletion rights for now? i.e not access:categories, for example?

@adriaaaa
Copy link
Author

@eastandwestwind that is correct. The DRP spec hasn't defined the access:categories and access:specific schema or sematics yet, see this github issue

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants