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

Add support for security policies in resource directory #259

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pulsastrix
Copy link

@pulsastrix pulsastrix commented Oct 15, 2021

This PR extends the current resource directory implementation in aiocoap/cli/rd.py with basic support for security policies, making it possible to restrict access to specific endpoints, which can be useful for a number of use cases:

  • Environments where clients expect a specific endpoint to be registered under a pre-defined name
  • Environments where some registrations should be invisible to some clients based on their credentials

The configuration happens using a separate JSON file that needs to be provided with the --security-policy command line option.
The file format is currently described in the rd.py module description, a CDDL description (like in credentials.cddl) is not included yet (but should be added before merging). included in aiocoap/cli/securitypolicy.cddl.

This is my first pull request to this repository, I am open to feedback if there is anything I can improve.

This allows users of the resource directory implementation to define
policies policies, i.e. limit access to specific endpoint names and sectors.

The configuration happens using a separate JSON file that needs to be
provided with the `--security-policy` command line option.

The file format is currently described in the rd.py module description, a
CDDL description (like in credentials.cddl) is not included yet.
@pulsastrix pulsastrix marked this pull request as ready for review October 28, 2021 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant