mautrix / slack

A Matrix-Slack puppeting bridge
GNU Affero General Public License v3.0
61 stars 17 forks source link

Slack avatars not bridged reliably #12

Open CyberShadow opened 1 year ago

CyberShadow commented 1 year ago

After setting up the bridge, I've noticed that only some of the bridged Slack users have avatars.

Not sure what the pattern is.

slagiewka commented 11 months ago

I did some digging:

db=# select avatar, avatar_url, avatar_set from puppet where avatar_url = '' or avatar = '';
 avatar | avatar_url | avatar_set 
--------+------------+------------
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | f
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | t
        |            | f
        |            | t
        |            | f
        |            | t
(21 rows)

It seems that for some reason, there are entries for other users (puppets) that have avatar_set = TRUE while not having any links there. One occurrence of FALSE is where I set it manually. The other two occurrences are some weird puppets that don't even have name set.

I went with:

update puppet set avatar_set = FALSE where avatar = '' and avatar_url = '';

And then I changed the config for user display. A friend wrote to me, their name's changed and now I have

db=# select avatar, avatar_url, avatar_set from puppet where avatar_url = '' or avatar = '';
 avatar | avatar_url | avatar_set 
--------+------------+------------
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | f
        |            | t <--
(21 rows)

So there's been an update to the row, but not to the avatar details.