signal icon indicating copy to clipboard operation
signal copied to clipboard

Update profile avatar paths & hashes properly

Open AndrewFerr opened this issue 1 year ago • 1 comments

This is a leftover commit from https://github.com/mautrix/signal/pull/449 that may still be relevant.

When running the bridge without this change, avatar updates weren't getting saved to the store when I expected them to & new avatars wouldn't appear in Matrix until a bridge restart.

AndrewFerr avatar Mar 07 '24 18:03 AndrewFerr

As explained here, the problem this PR was thought to fix is no longer present. So that makes this PR just an optional performance change.

Other than avoiding an unneeded hash calculation, it also ensures that changes to a puppet's avatar path are saved to the DB when the puppet's profile metadata comes from a Signal profile (not a contact) and the profile avatar path changed but the hash did not.

AndrewFerr avatar Mar 12 '24 14:03 AndrewFerr