Marcus Rueckert
8778bdbcb5
Please note that this version of Synapse drops support for
PostgreSQL 11 and 12. The minimum version of PostgreSQL supported
is now version 13.
- Deprecations and Removals
- Remove support for PostgreSQL 11 and 12. Contributed by @clokep. (#18034)
- Features
- Added the email.tlsname config option. This allows specifying
the domain name used to validate the SMTP server's TLS
certificate separately from the email.smtp_host to connect
to. (#17849)
- Module developers will have access to the user ID of the
requester when adding check_username_for_spam callbacks to
spam_checker_module_callbacks. Contributed by
Wilson@Pangea.chat. (#17916)
- Add endpoints to the Admin API to fetch the number of invites
the provided user has sent after a given timestamp, fetch the
number of rooms the provided user has joined after a given
timestamp, and get report IDs of event reports against a
provided user (i.e. where the user was the sender of the
reported event). (#17948)
- Support stable account suspension from MSC3823. (#17964)
- Add macaroon_secret_key_path config option. (#17983)
- Bugfixes
- Fix bug when rejecting withdrew invite with a
third_party_rules module, where the invite would be stuck for
the client. (#17930)
- Properly purge state groups tables when purging a room with
the Admin API. (#18024)
- Fix a bug preventing the admin redaction endpoint from
OBS-URL: https://build.opensuse.org/package/show/network:messaging:matrix/matrix-synapse?expand=0&rev=360
72 lines
2.9 KiB
Diff
72 lines
2.9 KiB
Diff
From d8917666d6198873bca140c3c511ae230ee698ec Mon Sep 17 00:00:00 2001
|
|
From: Jan Zerebecki <jan.suse@zerebecki.de>
|
|
Date: Mon, 30 Aug 2021 17:31:31 +0200
|
|
Subject: [PATCH] Fix instert of duplicate key into event_json
|
|
|
|
When an incoming event id is present in event_json but not in events
|
|
synapse fails trying to insert it with "psycopg2.errors.UniqueViolation:
|
|
duplicate key value violates unique constraints", because it is only
|
|
filtered based on those that are in events.
|
|
|
|
I don't know why those become out of sync, but this happening was
|
|
reported by others before.
|
|
|
|
Fix this by using an upsert (which inserts or updates existing records)
|
|
instead of a normal insert.
|
|
|
|
Please verify that this is the safe and correct thing to do before
|
|
merging this. Verify e.g. that it doesn't allow breaking history
|
|
integrity or something like it. As I don't know enough to understand
|
|
what this change entails.
|
|
|
|
Fixes: https://github.com/matrix-org/synapse/issues/10718
|
|
Signed-off-by: Jan Zerebecki <jan.suse@zerebecki.de>
|
|
---
|
|
changelog.d/10719.bugfix | 1 +
|
|
synapse/storage/databases/main/events.py | 22 +++++++++++-----------
|
|
2 files changed, 12 insertions(+), 11 deletions(-)
|
|
create mode 100644 changelog.d/10719.bugfix
|
|
|
|
diff --git a/changelog.d/10719.bugfix b/changelog.d/10719.bugfix
|
|
new file mode 100644
|
|
index 00000000000..d928f74f6bf
|
|
--- /dev/null
|
|
+++ b/changelog.d/10719.bugfix
|
|
@@ -0,0 +1 @@
|
|
+Fix instert failure because of duplicate key when an incoming event id is present in the table event_json but not in events.
|
|
diff --git a/synapse/storage/databases/main/events.py b/synapse/storage/databases/main/events.py
|
|
index 40b53274fb3..830af72d5e6 100644
|
|
--- a/synapse/storage/databases/main/events.py
|
|
+++ b/synapse/storage/databases/main/events.py
|
|
@@ -1334,19 +1334,19 @@ def get_internal_metadata(event):
|
|
|
|
return im
|
|
|
|
- self.db_pool.simple_insert_many_txn(
|
|
+ self.db_pool.simple_upsert_many_txn(
|
|
txn,
|
|
table="event_json",
|
|
- values=[
|
|
- {
|
|
- "event_id": event.event_id,
|
|
- "room_id": event.room_id,
|
|
- "internal_metadata": json_encoder.encode(
|
|
- get_internal_metadata(event)
|
|
- ),
|
|
- "json": json_encoder.encode(event_dict(event)),
|
|
- "format_version": event.format_version,
|
|
- }
|
|
+ key_names=["event_id"],
|
|
+ key_values=[[event.event_id] for event, _ in events_and_contexts],
|
|
+ value_names=["room_id", "internal_metadata", "json", "format_version"],
|
|
+ value_values=[
|
|
+ [
|
|
+ event.room_id,
|
|
+ json_encoder.encode(get_internal_metadata(event)),
|
|
+ json_encoder.encode(event_dict(event)),
|
|
+ event.format_version,
|
|
+ ]
|
|
for event, _ in events_and_contexts
|
|
],
|
|
)
|