Valere Fedronic comments

Results 94 comments of


                                            Valere Fedronic

Posthog: Add more properties to unable to decrypt errors to allow more filtering options

Closed https://github.com/element-hq/element-meta/issues/2303 and do this task instead

Posthog: Add more properties to unable to decrypt errors to allow more filtering options

For UTD that are not resolved after the max_time, we might want to report `timeToDecypt=-1` That would allow to filter more easilly

Posthog: Add more properties to unable to decrypt errors to allow more filtering options

yes let's close there is a roundoff for EX

Expected UTDs: Hide UTDs for messages sent while you were not in the room ("Pre-join UTDs")

> I'm really confused by this title. Is this specific to pre-invite UTDs, or does it also cover UTDs sent between the invite and the join? "when room allows" ---...

Expected UTDs: Hide UTDs for messages sent while you were not in the room ("Pre-join UTDs")

> > If you are invited in an invite only room > > what do you mean by an invite only room? > > > I am not sure if...

Key backups: some backups have incorrect MAC causing UTDs on login

@kegsay FWIW if you try to decrypt a room key with the wrong key you will get this same error `InvalidPadding(UnpadError)` So it could also be a problem with a...

Key backups: some backups have incorrect MAC causing UTDs on login

> We're going to return a more sensible error once [matrix-org/matrix-rust-sdk#3250](https://github.com/matrix-org/matrix-rust-sdk/pull/3250) is merged. The error looks to be now ``` Failed to decrypt megolm session from backup The MAC of...

Posthog: Stop double reporting UTDs events when app is relaunched.

The graph we are focusing on are Unique UTD, so stop sending double reports won't impact the graphs.

Posthog: Stop double reporting UTDs events when app is relaunched.

Related https://github.com/element-hq/element-meta/issues/2332 @kegsay We depriosrised because we only focus on unique errors. But double reporting will be annoying if we add the new properties. For example a permanent UTD would...

Clients may throw away one-time keys which are still published on the server, or have messages in flight

Rust SDK is keeping a lot more OTK [(50 * 100)](https://github.com/matrix-org/vodozemac/blob/8d0e0f704adeaeacd1cfd66c59ca319e3f1fe016/src/olm/account/one_time_keys.rs#L45). Lib olm appears to keep 100. So this would be a lot less common on rust.