Merge pull request #5767 from matrix-org/rav/redactions/cross_room_id

log when a redaction attempts to redact an event in a different room
This commit is contained in:
Richard van der Hoff 2019-07-25 18:49:56 +01:00 committed by GitHub
commit 26d742fed6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 28 additions and 0 deletions

1
changelog.d/5767.bugfix Normal file
View File

@ -0,0 +1 @@
Log when a redaction attempts to redact an event in a different room.

View File

@ -271,6 +271,14 @@ class EventsWorkerStore(SQLBaseStore):
) )
continue continue
if original_event.room_id != entry.event.room_id:
logger.info(
"Withholding redaction %s of event %s from a different room",
event_id,
redacted_event_id,
)
continue
if entry.event.internal_metadata.need_to_check_redaction(): if entry.event.internal_metadata.need_to_check_redaction():
original_domain = get_domain_from_id(original_event.sender) original_domain = get_domain_from_id(original_event.sender)
redaction_domain = get_domain_from_id(entry.event.sender) redaction_domain = get_domain_from_id(entry.event.sender)
@ -639,9 +647,21 @@ class EventsWorkerStore(SQLBaseStore):
if not redaction_entry: if not redaction_entry:
# we don't have the redaction event, or the redaction event was not # we don't have the redaction event, or the redaction event was not
# authorized. # authorized.
logger.debug(
"%s was redacted by %s but redaction not found/authed",
original_ev.event_id,
redaction_id,
)
continue continue
redaction_event = redaction_entry.event redaction_event = redaction_entry.event
if redaction_event.room_id != original_ev.room_id:
logger.debug(
"%s was redacted by %s but redaction was in a different room!",
original_ev.event_id,
redaction_id,
)
continue
# Starting in room version v3, some redactions need to be # Starting in room version v3, some redactions need to be
# rechecked if we didn't have the redacted event at the # rechecked if we didn't have the redacted event at the
@ -653,8 +673,15 @@ class EventsWorkerStore(SQLBaseStore):
redaction_event.internal_metadata.recheck_redaction = False redaction_event.internal_metadata.recheck_redaction = False
else: else:
# Senders don't match, so the event isn't actually redacted # Senders don't match, so the event isn't actually redacted
logger.debug(
"%s was redacted by %s but the senders don't match",
original_ev.event_id,
redaction_id,
)
continue continue
logger.debug("Redacting %s due to %s", original_ev.event_id, redaction_id)
# we found a good redaction event. Redact! # we found a good redaction event. Redact!
redacted_event = prune_event(original_ev) redacted_event = prune_event(original_ev)
redacted_event.unsigned["redacted_by"] = redaction_id redacted_event.unsigned["redacted_by"] = redaction_id