Skip to content

Add an Admin API endpoint to redact all a user's events #17506

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.

Sign up for GitHub

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

Merged
anoadragon453 merged 15 commits into from
Sep 18, 2024
Merged

Add an Admin API endpoint to redact all a user's events #17506

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
18f369d
add admin API to redact all messages of user
H-Shay Jul 30, 2024
9a7a304
tests
H-Shay Jul 30, 2024
2eb245b
newsfragment
H-Shay Jul 30, 2024
13312d2
make reaction resumable and api asynchronous
H-Shay Aug 1, 2024
17d782a
update tests
H-Shay Aug 1, 2024
81a1f2b
update redaction process
H-Shay Aug 20, 2024
455198e
update docs and response
H-Shay Aug 20, 2024
d32bc95
tests
H-Shay Aug 20, 2024
9ff860a
requested changes
H-Shay Aug 29, 2024
24afcd8
requested changes
H-Shay Sep 11, 2024
dcb156c
Merge branch 'develop' into shay/admin_redact
H-Shay Sep 11, 2024
f8b055e
lint
H-Shay Sep 11, 2024
c92ca2d
requested changes
H-Shay Sep 12, 2024
22af98d
requested changes
H-Shay Sep 17, 2024
f1ac8f5
Small wording changes
anoadragon453 Sep 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/admin_api/user_admin_api.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1440,4 +1440,4 @@ The following fields are returned in the JSON response body:
- failed_redactions: a dict where the keys are event ids the process was unable to redact, if any, and the values are
the corresponding error that caused the redaction to fail

_Added in Synapse 1.115.0._
_Added in Synapse 1.116.0._
2 changes: 1 addition & 1 deletion synapse/handlers/admin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -451,7 +451,7 @@ async def _redact_all_events(
if event.type == "m.room.member":
content = event.content
if content:
if content.get("membership") == "Membership.JOIN":
if content.get("membership") == "join":
Comment thread
H-Shay marked this conversation as resolved.
Outdated
pass
else:
continue
Expand Down
20 changes: 13 additions & 7 deletions tests/rest/admin/test_user.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5117,7 +5117,8 @@ def test_redact_messages_all_rooms(self) -> None:
"""
Test that request to redact events in all rooms user is member of is successful
"""
# create rooms, some with updated redaction rules
# create rooms - room versions 11+ store the `redacts` key in content while
# earlier ones don't so we use a mix of room versions
rm1 = self.helper.create_room_as(
self.admin, tok=self.admin_tok, room_version="7"
)
Comment thread
H-Shay marked this conversation as resolved.
Outdated
Expand All @@ -5129,7 +5130,8 @@ def test_redact_messages_all_rooms(self) -> None:
# join rooms, send some messages
originals = []
for rm in [rm1, rm2, rm3]:
self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
join = self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
originals.append(join["event_id"])
for i in range(15):
event = {"body": f"hello{i}", "msgtype": "m.text"}
res = self.helper.send_event(
Expand Down Expand Up @@ -5179,7 +5181,8 @@ def test_redact_messages_specific_rooms(self) -> None:

originals = []
for rm in [rm1, rm2, rm3]:
self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
join = self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
originals.append(join["event_id"])
for i in range(15):
event = {"body": f"hello{i}", "msgtype": "m.text"}
res = self.helper.send_event(
Expand Down Expand Up @@ -5214,8 +5217,8 @@ def test_redact_messages_specific_rooms(self) -> None:
and event["redacts"] == event_id
):
matches.append((event_id, event))
# we redacted 15 messages
self.assertEqual(len(matches), 15)
# we redacted 16 messages
self.assertEqual(len(matches), 16)

channel = self.make_request(
"GET", f"rooms/{rm2}/messages?limit=50", access_token=self.admin_tok
Expand All @@ -5238,7 +5241,9 @@ def test_redact_status(self) -> None:

originals = []
for rm in [rm1, rm2, rm3]:
self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
join = self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
if rm == rm2:
originals.append(join["event_id"])
Comment thread
H-Shay marked this conversation as resolved.
Outdated
for i in range(5):
event = {"body": f"hello{i}", "msgtype": "m.text"}
res = self.helper.send_event(
Expand Down Expand Up @@ -5307,7 +5312,8 @@ def test_admin_redact_works_if_user_kicked_or_banned(self) -> None:

originals = []
for rm in [rm1, rm2, rm3]:
self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
join = self.helper.join(rm, self.bad_user, tok=self.bad_user_tok)
originals.append(join["event_id"])
for i in range(5):
event = {"body": f"hello{i}", "msgtype": "m.text"}
res = self.helper.send_event(
Expand Down