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

Implement nested sections #1

Open
Sidnioulz opened this issue Oct 10, 2014 · 0 comments
Open

Implement nested sections #1

Sidnioulz opened this issue Oct 10, 2014 · 0 comments
Assignees
Milestone

Comments

@Sidnioulz
Copy link
Collaborator

Some compositors implement modes or multiple environments and would like different policies for such environments.

Mandatory design/code changes:

  • add way to change compositor name or much better to provide/retrieve it at permission granting time
  • explore methods to prevent race conditions on compositor identity when granting permissions
  • explore methods to prevent delegating statefulness management to compositors to avoid errors; find a clear method to guarantee the validity of an access decision when it is received by the compositor

Two appraches at least:

1. get compositors to advertise new names and use a section per mode

  • No added complexity to policy reading
  • But added length: harder to author meaningful policies (must copy/paste additions in each section) and even more so to edit them programmatically (are sections identical by coincidence or because they should be sync'd?).
  • Trivial code changes

Changes to codebase:
ø

2. implement subsections

  • More compact, faster to edit for experts
  • Less readability within file
  • Might require editing the parser substantially
  • Added complexity throughout whole backend, might need to change get_permission to expose contextual cues on compositor identity

Changes to codebase:

  • add a subsection syntax for the files
  • make parser understand subsections
  • rework weston_config_get_section* functions
@Sidnioulz Sidnioulz self-assigned this Oct 10, 2014
@Sidnioulz Sidnioulz added this to the alpha milestone Oct 10, 2014
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