Skip to content

MSC2244: Mass redactions #2244

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
turt2live merged 10 commits into from
Nov 12, 2019
Merged

MSC2244: Mass redactions #2244

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ec38013
Proposal to allow multiple targets for one redaction event
tulir Aug 23, 2019
984e0af
Re-word auth rule section on handling each target separately
tulir Aug 24, 2019
cd75d0f
Fix authenticity/authorization terminology
tulir Aug 25, 2019
238b78b
Add potential issue with redacted_because field
tulir Aug 25, 2019
79a5663
Fix typos, inline links and move image into tree
tulir Aug 25, 2019
c909a7c
Move omitting redacted_because into proposal and add security conside…
tulir Aug 29, 2019
e6f85ca
Specify that "existing auth rules" means room v5
tulir Aug 30, 2019
7ba4564
Remove soft fail auth rule option
tulir Aug 31, 2019
b2ce6f8
Merge branch 'master' into mass_redactions
tulir Aug 31, 2019
30106aa
Add section about backwards compatibility
tulir Nov 4, 2019
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
Prev Previous commit
Next Next commit
Fix authenticity/authorization terminology 
Co-authored-by: Kitsune Ral <[email protected]>
Signed-off-by: Tulir Asokan <[email protected]>
  • Loading branch information
@tulir @KitsuneRal
tulir and KitsuneRal committed Aug 28, 2019
commit cd75d0f220e8fa2d64d19e04a21a6e4b4bb35854
10 changes: 5 additions & 5 deletions proposals/2244-mass-redactions.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,12 @@ redaction event.

#### Handle each target separately
The target events of an `m.room.redaction` shall no longer be considered when
deciding the authenticity of an `m.room.redaction` event. Any other existing
rules remain unchanged.
authorizing of an `m.room.redaction` event. Any other existing rules remain
unchanged.

When a server accepts an `m.room.redaction` using the modified auth rules, it
evaluates targets individually for authenticity under the existing auth rules.
Servers MUST NOT include failing and unknown entries to clients.
evaluates individually whether each target can be redacted under the existing
auth rules. Servers MUST NOT include failing and unknown entries to clients.

> Servers do not know whether redaction targets are authorized at the time they
receive the `m.room.redaction` unless they are in possession of the target
Expand All @@ -63,7 +63,7 @@ When the implementation receives a belated target from an earlier
authorized.

> Servers should not send belated target events to clients if their redaction
was found to be authentic, as clients were not made aware of the redaction.
was found to be in effect, as clients were not made aware of the redaction.
That fact is also used to simply ignore unauthorized targets and send the
events to clients normally.

Expand Down