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

Update Security and Privacy Considerations (2) #686

Merged
merged 5 commits into from
Feb 10, 2022

Conversation

mmccool
Copy link
Contributor

@mmccool mmccool commented Jan 31, 2022

Resolves #672

WIP: Do not merge, still being reviewed by Security TF

  • add trusted environment risk

Preview | Diff

@mmccool
Copy link
Contributor Author

mmccool commented Jan 31, 2022

To do: fix cross-references, href syntax is wrong, should set it up also so section names and numbers get filled in by respec.

@mmccool mmccool self-assigned this Feb 3, 2022
Copy link

@j1y3p4rk j1y3p4rk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am also fine with the changes.

@@ -1158,6 +1160,14 @@ <h1>System Topologies (Horizontals)</h1>
They are not exhaustive and are presented to motivate for the WoT architecture and building blocks
that are defined in later sections of this specification.
</p>
<p>
This section also makes use of the concept of a Trusted Environment,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggest to create a terminology entry for "Trusted Environment"

@mlagally mlagally marked this pull request as ready for review February 10, 2022 11:28
@mlagally mlagally merged commit d5dae9e into w3c:main Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review Security and Privacy Considerations
4 participants