You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, when given a service user with sufficient permissions to do this, users who are not in the set organization/project will also be subjected to the sync, and therefore usually simply deleted. This is obviously very hazardous behavior, we should not just do that.
To limit the sync to just one organization is pretty simple:
However, limiting this to just the project grants is a bit more complex. This information is not tied to the user, but the project, so at the very least we would need an additional query for every user to check which projects they are in (i.e., search for the user's ID with https://zitadel.com/docs/apis/resources/mgmt/management-service-list-project-grant-members, or get a list of users with that search and add their IDs to the AndQuery - actually, the latter just seems better, hopefully there is not a limit to the number of IDs that can be added to a query).
These limits also add an issue in cases where an existing user is added to a new project, since it will create a new user instead of simply adding a new scope; this is especially problematic since we made the Zitadel ID a foreign key. I believe that that should never be the case in a famedly context, but we should confirm this.
The text was updated successfully, but these errors were encountered:
Currently, when given a service user with sufficient permissions to do this, users who are not in the set organization/project will also be subjected to the sync, and therefore usually simply deleted. This is obviously very hazardous behavior, we should not just do that.
To limit the sync to just one organization is pretty simple:
However, limiting this to just the project grants is a bit more complex. This information is not tied to the user, but the project, so at the very least we would need an additional query for every user to check which projects they are in (i.e., search for the user's ID with https://zitadel.com/docs/apis/resources/mgmt/management-service-list-project-grant-members, or get a list of users with that search and add their IDs to the
AndQuery
- actually, the latter just seems better, hopefully there is not a limit to the number of IDs that can be added to a query).These limits also add an issue in cases where an existing user is added to a new project, since it will create a new user instead of simply adding a new scope; this is especially problematic since we made the Zitadel ID a foreign key. I believe that that should never be the case in a famedly context, but we should confirm this.
The text was updated successfully, but these errors were encountered: