Skip to content

Comments

Update README.md#7

Open
ewwollesen wants to merge 2 commits intocoltoneshaw:mainfrom
ewwollesen:main
Open

Update README.md#7
ewwollesen wants to merge 2 commits intocoltoneshaw:mainfrom
ewwollesen:main

Conversation

@ewwollesen
Copy link
Contributor

Added a query to swap the Id's of two users.

Added a query to swap the Id's of two users
@coltoneshaw
Copy link
Owner

@ewwollesen so for this we would want to have the second user take full ownership of the first users data?

What context was this used? I wonder if we need to purge sessions too, just to be safe.

Added a note to revoke sessions of the users.
@ewwollesen
Copy link
Contributor Author

@ewwollesen so for this we would want to have the second user take full ownership of the first users data?

What context was this used? I wonder if we need to purge sessions too, just to be safe.

This is for cases when they are using Email as the IdAttribute in SAML, and a user's email gets changed for some reason (getting married, for example). When it gets changed on the SAML side and they log in, it will create a new account. So this just swaps the old account with the new account.

That's not a bad idea though to revoke their sessions too first. I added a note to recommend that.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants