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

Implement event redaction #768

wants to merge 17 commits into
base: master


Copy link

commented Jul 27, 2019

Fixes #620.

This PR also includes a minor database fix taken from #696.

This PR doesn't provide full check on redactor's power level yet, which should be fixed at some point later, together with Dendrite's other power level TODOs. Another aspect in which it can be improved is that some similar code appearing in both roomserver and syncapi may be able to get factored out somewhere.

Signed-off-by: Alex Chen

Pull Request Checklist

  • I have added any new tests that need to pass to testfile as specified in docs/
  • Pull request includes a sign off

Cnly added some commits Jul 27, 2019

Implement event redaction
Signed-off-by: Alex Chen <>
Fix arguments in selectEventsSQL
Signed-off-by: Alex Chen <>
Update gomatrixserverlib to v0.0.0-20190727114627-340519e0d9e3
Signed-off-by: Alex Chen <>
Implement POST version of /redact
Signed-off-by: Alex Chen <>
Add more sanity checks
Signed-off-by: Alex Chen <>
Variable and type clarifications
Signed-off-by: Alex Chen <>
Add missing license header
Signed-off-by: Alex Chen <>
Ignore error from rows.Close()
Signed-off-by: Alex Chen <>
Remove unnecessary sanity checks
Signed-off-by: Alex Chen <>
Add docs for validateRedactions
Signed-off-by: Alex Chen <>

@Cnly Cnly marked this pull request as ready for review Aug 1, 2019

Merge branch 'master' into event-redaction
Signed-off-by: Alex Chen <>

@Cnly Cnly requested a review from anoadragon453 Aug 1, 2019

Cnly added some commits Aug 3, 2019

Add newly passing test
Signed-off-by: Alex Chen <>
Remove obsolete TODOs
Signed-off-by: Alex Chen <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
1 participant
You can’t perform that action at this time.