element-hq / element-meta

Shared/meta documentation and project artefacts for Element clients
75 stars 12 forks source link

New posthog codes for new decryption failure reasons #2541

Closed richvdh closed 3 weeks ago

richvdh commented 1 month ago

We've recently added new decryption failure reasons; in particular:

We should add new PostHog reason codes for these failure reasons, and wire up the applications to send them correctly.

Not strictly the same cause but may be trivial to fix at the same time:

BillCarsonFr commented 1 month ago

I created an issue on rust https://github.com/matrix-org/matrix-rust-sdk/issues/4048 that is covering the work on rust I think. Linking here so we can discuss it

richvdh commented 1 month ago

I created an issue on rust matrix-org/matrix-rust-sdk#4048 that is covering the work on rust I think.

That issue lays some groundwork, but does not update the posthog integration in the rust sdk

uhoreg commented 1 month ago

matrix-analytics-events PR: https://github.com/matrix-org/matrix-analytics-events/pull/114

uhoreg commented 1 month ago

@BillCarsonFr is doing the EX bits, and I'm doing the EW bits

uhoreg commented 4 weeks ago

React SDK PR: https://github.com/element-hq/element-web/pull/28292 (in draft until tests written)

BillCarsonFr commented 3 weeks ago

Not strictly the same cause but trivial to fix at the same time:

No trivial, need to be handled separatly

richvdh commented 3 weeks ago

I've moved this to the "Trust & Decoration" workstream, since the codes affected here will only affect devices with the "Exclude insecure devices when sending/receiving messages" labs option enabled, for now.