Skip to content
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

Add config to disallow manual double puppeting #1

Merged
merged 3 commits into from Feb 16, 2024

Conversation

AndrewFerr
Copy link
Member

as some instances may want to discourage users from sharing their Matrix access tokens.

Disabling manual double puppeting still allows automatic double puppeting for any server configured with a login shared secret.

If both manual and automatic double puppeting are enabled for a user, allow login-matrix to replace the server-managed Matrix token with the one given by the command.

Also add the refresh-matrix command to fix potential issues with automatically-managed double puppet sessions.

as some instances may want to discourage users from sharing their Matrix
access tokens.

Disabling manual double puppeting still allows automatic double
puppeting for any server configured with a login shared secret.

If both manual and automatic double puppeting are enabled for a user,
allow `login-matrix` to replace the server-managed Matrix token with
the one given by the command.

Also add the `refresh-matrix` command to fix potential issues with
automatically-managed double puppet sessions.
@AndrewFerr AndrewFerr self-assigned this Feb 6, 2024
@AndrewFerr AndrewFerr requested a review from a team as a code owner February 6, 2024 13:29
Copy link
Member

@Half-Shot Half-Shot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems okay to me 👍

Notably, disallow (but don't hide) manual double puppeting commands for
users whose DP tokens are managed automatically.  This averts some edge
cases that could happen otherwise.
@AndrewFerr AndrewFerr merged commit 4c19fa2 into element-master Feb 16, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants