-
Notifications
You must be signed in to change notification settings - Fork 158
adr: Add thoughts about guest users #2183
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
Open
rhafer
wants to merge
1
commit into
opencloud-eu:main
Choose a base branch
from
rhafer:issue/1540
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,192 @@ | ||
| --- | ||
| title: "Guest users in OpenCloud" | ||
| --- | ||
|
|
||
| * Status: pending | ||
| * Deciders: [] | ||
| * Date: 2026-01-20 | ||
|
|
||
| Reference: https://github.com/opencloud-eu/opencloud/issues/2111 | ||
|
|
||
| ## Context and Problem statement | ||
|
|
||
| To allow collaboration with external Users (Users that don't yet have an | ||
| account in the IDP, and might be external to the organization), it should | ||
| be possible to invite "Guest Users" into and OpenCloud instance. | ||
|
|
||
| ## Requirements | ||
|
|
||
| - the audit trail of the external user accessing the resource needs to | ||
| be maintained, that means sharing via a password protected public link | ||
| is not sufficient as access to that one is tracked as if the creator | ||
| of the link accessed the resource | ||
| - external users need to be authenticated just like "normal" users, when | ||
| accessing the shared resource (including the possibility to use 2FA) | ||
| - the ability to invite external users is tied to a separate permission | ||
| (e.g. "can invite guest users") | ||
| - make it work with all (most) of the user-management configurations we support | ||
| - avoid creating "Shadow IT" Infrastructure, e.g. we don't want to | ||
| create/maintain a separate IDP instance just for Guest User that would | ||
| allow bypassing corporate rules for Identity Management | ||
|
|
||
| ## Questions still to be answered | ||
|
|
||
| - what's the life cycle of a guest user? | ||
| - Who's responsible for deprovisioning? | ||
| - Do guest users expire after a certain time? | ||
| - Do we need to keep track of who invited whom and when? (not just in | ||
| the audit log?) | ||
| - who can see the list of guest users? | ||
| - once a guest user is created, is everyone in the organization able to | ||
| share with that guest user? | ||
| - what are guest user allowed to do? (are they able to share, lookup | ||
| other users?) | ||
| - should this be really tied just to creating shares? Or should we have | ||
| a more generic "invite user" feature that can be used in other | ||
| contexts as well? | ||
| - What if the user already exists but used a different mail address in | ||
| his account (e.g. sub-addressing?). | ||
|
|
||
| ## Obstacles | ||
|
|
||
| ### UserIDs | ||
|
|
||
| - Every user in OpenCloud needs to have a userid assigned | ||
| - Sharing, as many other features, needs that userid for storing the | ||
| share (share service) and for assigning the grants on the shared | ||
| resources (storage provider) | ||
| - When an external IDP is used the generation of that userid is usually | ||
| not in control of OpenCloud (exception User-Autoprovisioning, or when | ||
| the Provisioning/Education API is used), but where taking the userid | ||
| from some LDAP Attribute maintained in the external system | ||
|
|
||
| ### Lots of identity management options | ||
|
|
||
| - OpenCloud provides many different ways to consume user-accounts. Guest | ||
| users are supposed to be working with all/most of them: | ||
| - Internal IDP, with internal LDAP service | ||
| - External IDP, with external LDAP service | ||
| - External IDP, with manual provisioning via the | ||
| Education/Provisioning APIs (to a local OpenCloud specific LDAP | ||
| service) - e.g. in multi-tenant setups | ||
| - External IDP, with User-Autoprovisioning (also to a local OpenCloud | ||
| specific LDAP service) | ||
| - everything in-between and outside of the above | ||
| - Each of these options have different ways for user-provisioning and in | ||
| the way userids are generated and managed | ||
|
|
||
| ### How do we keep track of invitations? | ||
|
|
||
| - Completely rely on external system? | ||
| - Track creation and acceptance of invitations somehow? | ||
| - Do invitation expire at some point? | ||
|
|
||
| ## Possible solutions | ||
|
|
||
| ### Re-vitalize the PoC implementation of the invitations service and finalize it (<https://github.com/opencloud-eu/opencloud/blob/main/services/invitations/README.md>) | ||
|
|
||
| - Implements parts of the MSGraph Invitation Specification | ||
| (<https://learn.microsoft.com/en-us/graph/api/resources/invitation?view=graph-rest-1.0>) | ||
| - Currently there's just a single backend that allows creating users, | ||
| using the Keycloak Admin API | ||
| - As part of the user creation keycloak triggers an email to be sent to | ||
| the invited user to get him to verify his email address and set a | ||
| password. This is not really and invitation email. | ||
|
|
||
| #### Pros | ||
|
|
||
| - A partial implementation already exists | ||
| - no shadow IT | ||
|
|
||
| #### Cons | ||
|
|
||
| - while the emails sent by keycloak can be themed. There is no way | ||
| to add custom content, like: "you've being invited by user X to | ||
| access resource Y" | ||
| - the keycloak admin API does not return the password reset link in | ||
| the response, so we can't use that to send a custom email | ||
| - the keycloak implementation is not a real "user invitation" | ||
| workflow, the user experience for the invited user is not ideal | ||
| - The workflow likely only works with a limited set of setups. | ||
| (Specifically: a keycloak that is able to write into a connected | ||
| LDAP database, that OpenCloud can consume) | ||
| - As the invitations are not really tracked, e.g. we don't really | ||
| "know" if an invitation was accepted | ||
| - Requires direct access to the Identity Management System | ||
|
|
||
| ### Invitation Service + support for pending shares in the share manager | ||
|
|
||
| - Create some form in invitation manager and provide tools/documentation | ||
| for customers to hook that up with their Identity Management System | ||
| - User's with the "right" privileges are able to create invitations, | ||
| invitations get a unique identifier. Other data maintained on the | ||
| invitation: | ||
| - Invited user's email address | ||
| - Invited user's userid (once the user account was provisioned) | ||
| - Inviting user's userid | ||
| - Creation timestamp | ||
| - Invitation State (Pending, Accepted, …) | ||
| - (more probably) | ||
| - our sharing API | ||
| ('graph/v1beta1/drives/{drive-id}/items/{item-id}/invite') is enhanced | ||
| to allow creating shares that target an invitation as the share | ||
| recipient. (That share would only be persisted in the 'shares' service | ||
| and would not yet crate any grants on the filesystem, or send out | ||
| sharing notifications). (Requires changes to the CS3 sharing APIs) | ||
| - A middleware (specific to the Identity Management System) is | ||
| "informed" (e.g. via web hooks or a message queue) when a new | ||
| invitation is created. That middleware is responsible for provisioning | ||
| the user account of the guest user. Whatever this process looks like | ||
| it completely up to the middleware (maybe it triggers some invitation | ||
| workflow or it could just even open a support ticket with the IDP | ||
| admin) | ||
| - once the user is provisioned the middleware calls back into our | ||
| invitations service,marks the invitation as "accepted" and provides | ||
| the "userid" of the guest user. The invitations service then triggers | ||
| the "pending" shares to be processes, which causes the filesystem | ||
| grants to be written and notifications to be send out to the guest | ||
| user. | ||
| - We'd provide a reference implementation of that middleware, that works | ||
| with keycloak | ||
|
|
||
| #### Pros | ||
|
|
||
| - Agnostic to whatever Identity Management System is used | ||
| - We have an audittrail about who was invited by whom at what point | ||
| in time | ||
| - no shadow IT | ||
|
|
||
| #### Cons | ||
|
|
||
| - somewhat complex | ||
| - likely requires changes to the CS3 APIs | ||
|
|
||
| #### Implementation Obstacles | ||
|
|
||
| - Permissions on spaces are currently not tracked in the share | ||
| manager, the are purely managed via grants. So currently the share | ||
| manager service currently does not know anything about (invited) | ||
| users being assigned to spaces | ||
|
|
||
| ## Additional thoughts | ||
|
|
||
| If OpenCloud were responsible for allocating the UserIDs of all users | ||
|
Contributor
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. Good point! I was wondering also, if that could simplify things in the future and let us get rid of the LDAP dependency. |
||
| the solution sketch above would likely loose some of its complexity. We | ||
| would "roll" the userid for the invited user already when creating the | ||
| invite. And could skip the step of creating a "pending" Share with an | ||
| invitation assigned. As we have an ID already we could just create a | ||
| "normal" share an even populated the grants on the filesystem for that | ||
| share (or space) | ||
|
|
||
| We've been pondering on the idea of making OpenCloud manage all UserIDs | ||
| for quite a while as it would have some additional benefits for the | ||
| whole user management story. | ||
|
|
||
| - We wouldn't rely anymore on the external Identity Management system to | ||
| provide a unique id with certain properties. Ideally the only unique | ||
| thing we'd need from the external system is the `iss` and `sub` | ||
| claims of the IDP and those are required by the OIDC standards. | ||
|
|
||
| It could be worth to spend some time on figuring out a migration path | ||
| towards such a solution, before spending resources on a complex guest | ||
| features implementation. | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I agree with the requirements. Maybe we need to add that the process can be asynchonous and take some time because we are crossing a lot of boundaries between different systems.