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

App level encryption is not addressed. #111

Closed
SpencerDawkins opened this issue Apr 6, 2022 · 4 comments · Fixed by #119
Closed

App level encryption is not addressed. #111

SpencerDawkins opened this issue Apr 6, 2022 · 4 comments · Fixed by #119
Assignees

Comments

@SpencerDawkins
Copy link
Collaborator

From @evyncke

Based on the title it is unclear whether this section also covers "app layer encryption", I guess no but this should be made clear (perhaps high in section 7 by stating that this doc does not cover app layer encryption).

@SpencerDawkins
Copy link
Collaborator Author

This paragraph

In this document, we will focus on media encrypted at the transport layer, whether encrypted "hop-by-hop" or "end-to-end". Because media encrypted at the application layer will only be processed by application-level entities, this encryption does not have transport-layer implications.

appears later in the document - I can move this up under the bullet points and make that clearer.

@SpencerDawkins
Copy link
Collaborator Author

Fixed as in my previous comment.

@evyncke
Copy link

evyncke commented Apr 11, 2022

BTW, unsure how the authors want me to mark "I have read the change and I like it" ? Currently, simply using a "thum up" emoticon on the solution comment.

Let me know if you prefer another marking.

@SpencerDawkins
Copy link
Collaborator Author

@evyncke,

BTW, unsure how the authors want me to mark "I have read the change and I like it" ? Currently, simply using a "thum up" emoticon on the solution comment.

Let me know if you prefer another marking.

If I can't hear you either giggling or screaming all the way to Texas, and I see a "thumbs up" emoticon, that's clear enough to me!

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

Successfully merging a pull request may close this issue.

2 participants