Skip to content

Commit 7f734b9

Browse files
authored
Merge pull request cardstack#1441 from cardstack/cs-7011-chriss-matrix-bug-old-matrix-event-data-is-not-serialized-as
Add defensive coding for old unserialized matrix event content data
2 parents 77a90cf + e6e0725 commit 7f734b9

File tree

1 file changed

+8
-0
lines changed
  • packages/host/app/lib/matrix-handlers

1 file changed

+8
-0
lines changed

packages/host/app/lib/matrix-handlers/index.ts

+8
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,14 @@ export async function addRoomEvent(context: EventSendingContext, event: Event) {
6969
// Corresponding encoding is done in
7070
// sendEvent in the matrix-service
7171
if (event.content?.data) {
72+
if (typeof event.content.data !== 'string') {
73+
console.warn(
74+
`skipping matrix event ${
75+
eventId ?? stateKey
76+
}, event.content.data is not serialized properly`,
77+
);
78+
return;
79+
}
7280
event.content.data = JSON.parse(event.content.data);
7381
}
7482
eventId = eventId ?? stateKey; // room state may not necessary have an event ID

0 commit comments

Comments
 (0)