-
Notifications
You must be signed in to change notification settings - Fork 1
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
initial WebID Document considerations #9
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -118,3 +118,41 @@ The attacker writes a malicious `text/html` file to the server. Depending on the | |||||||||||||||||||||||||||||
#### Considerations | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
Servers are strongly encouraged to consider the countermeasures in the context of the use cases they want to enable or disable on a given storage. For instance, using `Content-Security-Policy: sandbox` will universally prohibit various functionalities for applications, including but not limited to accessing local storage, executing scripts, using forms, interacting with plugins, or including external content. This broad range of restrictions may not be desirable for various categories of applications that rely on client-side storage mechanisms, collaborative features, or dynamic content interaction. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
## Protecting WebID Document ## {#protecting-webid-document} | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
Solid access policy enforcement relies heavily on the identities of agents. Requesting Parties are identified | ||||||||||||||||||||||||||||||
by [[WebID]] and applications by ClientID [[Solid.OIDC]]. | ||||||||||||||||||||||||||||||
A User's WebID Document includes trust anchors, like designation of their [[Solid.OIDC]] Provider (also known as _issuer_). | ||||||||||||||||||||||||||||||
In other approaches, public keys could be published or discoverable via the user's WebID Document. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
### Fully impersonate the user | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
If a malicious app can write or append to the user's WebID Document, | ||||||||||||||||||||||||||||||
it could inject trust anchors, allowing complete impersonation. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
#### Exploit Prerequisites | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
* Solid's authorization system depends on a specific trust anchor in the user's WebID Document. | ||||||||||||||||||||||||||||||
For example, [[Solid.OIDC]] issuer designation or pubic keys/keyset. | ||||||||||||||||||||||||||||||
* User authorizes a malicious Solid application to write or append to their WebID Document. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
#### Attack | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
* The attacker publishes an attractive malicious application, such as one that generates cool AI avatars and setting them in the user's profile. | ||||||||||||||||||||||||||||||
* User authenticates with the malicious application. | ||||||||||||||||||||||||||||||
* If the access control system enforces client constraints, the user also authorizes the application to write or append to their WebID Document. | ||||||||||||||||||||||||||||||
* Application injects attacker's trust anchor into the user's WebID Document. | ||||||||||||||||||||||||||||||
* The attacker can now completely impersonate the user, with full access to all the data owned by the user, | ||||||||||||||||||||||||||||||
as well as access to any data shared with that user by others, to the full extent they granted to the impersonated user. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
### Countermeasures | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
* There is no requirement to expose WebID Document via [[Solid.Protocol]] nor to host it in Solid Storage. | ||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's needed if the end user wants to use a Solid app to update their photo and contact details. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I included it in the considerations section. I didn't mention There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
I very much agree that a dedicated predicate is needed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
And what happens if there is no rdfs:seeAlso in the WebID document? The user must manually request one? And suppose a different seeAlso is needed, does that go in the first rdfs:seeAlso and how many recursions are apps expected to go to find the seeAlsos in the seeAlsos? And if there are 10 seeAlso docs and the photo-album pointer is in the 10th - this means 10 loads instead of a maximum of one or two. There is no way for a user to prioritize public data by placing it where apps will see it first or second. Look at the diagrams I have posted above - with #2 (a dedicated predicated) there is a maximum of two loads to get to something prioritized. In #3, with only seeAlso, there is an unknown number of loads |
||||||||||||||||||||||||||||||
Service providers who offer hosting of WebID Documents can minimally support | ||||||||||||||||||||||||||||||
a set of discovery features that rely on the WebID document and provide a | ||||||||||||||||||||||||||||||
highly secured interface to change it, possibly requiring two-factor authentication. | ||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||
#### Considerations | ||||||||||||||||||||||||||||||
Users need a way to manage their social profiles, approaches like linking from the user's WebID Document to | ||||||||||||||||||||||||||||||
less protected vcard are available to accomplish it without opening discussed attack vectors. | ||||||||||||||||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What are you basing this performance aspect on? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. My suggestion doesn't have 3. ... So we have O(1) and O(2), if we have a list of n users we would have O(n) and O(2n). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...except that the WebID XG never published this, with or without a version declaration. It would be better to title it as what it was,
"WebID 1.0 Editor's Draft"
(which must be read and understood as"Editor's Draft of WebID 1.0"
) ... and of course, the document now served from that location says "Draft Community Group Report 05 March 2014", which is similarly but differently inaccurate and contrary to the "publisher" identified here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case we can't quickly resolve it in this PR I created a dedicated issue for it #10