Skip to content

Security considerations for integrity of solid:oidcIssuer triples #219

Open

Description

@Otto-AA

I am not sure if this is the correct specification, but I'd suggest to add security considerations for the write/changability of the solid:oidcIssuer triple in the WebID profile.

The solid:oidcIssuer is used to verify that access token are created by the right entity for a WebID:

A WebID Profile lists the OpenID Providers who are trusted to issue tokens on behalf of the agent who controls the WebID.

In turn this means, if someone is able to add or change this triple in the profile, they can point it to a server under their control and create valid access tokens. Thus, append or write access to the profile indirectly gives access to anything the webID has access to.

I propose to add a Security Consideration along the lines of: "WebID providers SHOULD/MUST ensure the integrity of solid:oidcIssuer triples. Any agent that can modify or add solid:oidcIssuer triples to the WebID could impersonate this WebID."

In practice, this could be implemented in one of following ways:

  • make the web ID readonly (I think this is what ESS does in combination with an extended profile)
  • reject requests that modify or add solid:oidcIssuer triples to the webID, even if the agent has append/write access

Activity

  1. changed the title [-]Security considerations for writable solid:oidcIssuer[/-] [+]Security considerations for integrity of solid:oidcIssuer triples[/+] on Mar 26, 2023
  2. elf-pavlik commented on Mar 27, 2023

    @elf-pavlik
    Member

    Thank you @Otto-AA, there is also https://github.com/solid/webid-profile but since OIDC relies on it we can also add it here. /cc @VirginiaBalseiro

  3. jeff-zucker commented on Apr 5, 2023

    @jeff-zucker
    Member

    I am in favor of your second but not the first. I find the ESS approach of forbidding apps from editing the profile document much too restrictive. OTOH, if the IdP allows patches to the profile but intercepts them and disallows changes to oidcIssuer, that would seem to allow both security and user control over their own profile.,

  4. csarven commented on May 21, 2024

    @csarven
    Member

    The issuer is in and itself a point of failure when it is compromised.

  5. elf-pavlik commented on May 29, 2024

    @elf-pavlik
    Member

    We plan to discuss this issue next week Special Topic Meetings (view) on Tuesday

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions