UnifierHQ / unifier

A cross-server and cross-platform bridge bot that works just as fast as you can type 🚀
https://wiki.unifierhq.org
GNU Affero General Public License v3.0
12 stars 17 forks source link

[Bug] Wrong user profile information used on bridged messages #185

Closed greeeen-dev closed 3 hours ago

greeeen-dev commented 3 hours ago

Checklist

Version

3.6.7

Bug description

In some messages, Unifier will display the wrong user information when bridging messages. This has only happened for messages sent from Revolt so far. Sender information is somehow stored just fine in the message cache and reloading the bridge seems to temporarily fix the problem.

Possible causes:

Going to get all hands on deck to investigate this.

Reproduction steps

  1. Send a message from Revolt (or Guilded)

Expected behavior

User profile appears properly for each message.

Plugins

Revolt Support, Guilded Support

Modified instance

Modified instance repository

No response

Screenshots

No response

lezetho commented 3 hours ago

hmm yes yes i see i shall take a look at home

greeeen-dev commented 3 hours ago

Decided to implement a new UnifierUser class for handling user names, avatars, etc. Hopefully this helps fix the bug.

greeeen-dev commented 3 hours ago

Closing this issue for now since I've created a possible patch but please do check my new code to see if the bug could happen again.