bug(collab): serialiseer Matrix-ops vóór verzending #1040

New issue

Closed

opened 2026-08-01 08:26:02 +00:00 by brenno · 1 comment

brenno commented

2026-08-01 08:26:02 +00:00

Owner

Impact
Een volger accepteert alleen version + 1. Bij ontvangst van v2,v1 wordt v2 definitief overgeslagen, v1 toegepast en divergeren de decks. WebDAV voorkomt dit al met een append-chain; Matrix niet.

Relevante code

lib/collab/collab_session_controller.dart:66
lib/collab/collab_session.dart:124
lib/collab/matrix_relay_transport.dart:106
lib/collab/webdav_async_transport.dart:81

Acceptatie

Uitgaande Matrix-ops behouden aanmaak-/versievolgorde, ook bij overlappende calls.
Een verzendfout breekt de keten voor latere ops niet stil en wordt beheerst gerapporteerd.
Regressietest vertraagt de eerste fake Matrix-PUT, laat een tweede eerst kunnen afronden en bewijst dat de guest toch v1 en v2 plus het volledige einddeck ontvangt.

Context CollabSessionController zet één deckwijziging om in meerdere geordende DeckOps. Op main worden de bijbehorende session.submit- en Matrix-sendOp-aanroepen niet geserialiseerd. De asynchrone encryptie en HTTP-PUTs kunnen versie 2 daardoor vóór versie 1 op de homeserver vastleggen. Impact Een volger accepteert alleen version + 1. Bij ontvangst van v2,v1 wordt v2 definitief overgeslagen, v1 toegepast en divergeren de decks. WebDAV voorkomt dit al met een append-chain; Matrix niet. Relevante code - lib/collab/collab_session_controller.dart:66 - lib/collab/collab_session.dart:124 - lib/collab/matrix_relay_transport.dart:106 - lib/collab/webdav_async_transport.dart:81 Acceptatie - Uitgaande Matrix-ops behouden aanmaak-/versievolgorde, ook bij overlappende calls. - Een verzendfout breekt de keten voor latere ops niet stil en wordt beheerst gerapporteerd. - Regressietest vertraagt de eerste fake Matrix-PUT, laat een tweede eerst kunnen afronden en bewijst dat de guest toch v1 en v2 plus het volledige einddeck ontvangt.

brenno added the

accepted

bug

labels

2026-08-01 08:26:02 +00:00

brenno referenced this issue from a commit

2026-08-01 19:12:12 +00:00

fix(collab): serialiseer uitgaande Matrix-ops vóór verzending (#1040)

brenno referenced this issue

2026-08-01 19:12:20 +00:00

fix(collab): serialiseer uitgaande Matrix-ops vóór verzending (#1040) #1086

brenno referenced this issue from a commit

2026-08-01 19:12:29 +00:00

Merge pull request 'fix(collab): serialiseer uitgaande Matrix-ops vóór verzending (#1040)' (#1086) from fix/collab-serialize-ops-1040 into main

brenno commented

2026-08-01 19:12:35 +00:00

Author

Owner

Opgelost op main (PR #1086, merge in e43ff81a). De relay-transport serialiseert uitgaande sends via een verzendketen; volgorde blijft behouden en een verzendfout breekt de keten niet. Regressietest parkeert de eerste op-PUT.

Opgelost op main (PR #1086, merge in e43ff81a). De relay-transport serialiseert uitgaande sends via een verzendketen; volgorde blijft behouden en een verzendfout breekt de keten niet. Regressietest parkeert de eerste op-PUT.

brenno closed this issue

2026-08-01 19:12:35 +00:00