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 OSCOAP to get security running #53

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

Conversation

jimsch
Copy link

@jimsch jimsch commented Jan 16, 2017

This is not really a request for doing a pull yet, it is more a case of asking for review to see if this is something that you are interested in and if you think that it is in the correct direction.

This is not a complete implementation yet. Some of the things that I know are missing:

  • Observe and blockwise transfer are going to require doing a cached exchange in the OSCOAP layer.
  • The OSCOAP option number has not been assigned so I have chosen something arbitrary (and wrong) for testing purposes.
  • Need to figure out all of the builds to make sure that all of them will build correctly. Part of this is going to be the question of how do do builds for both with and without OSCOAP in all probability given the set of components that I am dragging in for cryptography. Also there may be a problem as not all of the modules that I am currently using are signed and thus if this is signed then they cannot be loaded.

This gets basic OSCOAP working, but there is still alot to look at
before it is finalized
Add the code on ReceiveRequest to do replay detection.
Clean up build errors for when INCLUDE_OSCOAP is not defined.
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