search

matrix-org / matrix-appservice-irc

Node.js IRC bridge for Matrix
Apache License 2.0
462 stars 151 forks source link

Appservice doesn't track rooms correctly leading to incorrect kick #1587

Closed MTRNord closed 2 years ago

MTRNord commented 2 years ago

Describe the bug If an irc room is bridged to both a tombstoned room and the new one and a user leaves the tombstoned room the appservice will kick the user in the new room.

To Reproduce Steps to reproduce the behavior:

  1. Join mediawiki on libera (not the matrix.org alias.)
  2. Follow the tombstone
  3. Leave the libera room
  4. Appservice will kick you in the new room

Expected behavior Nothing happens in the new room.

Additional context It seems like the old room wasn't unvridged when tombstoned. Which probably is also wrong here 🤔

MTRNord commented 2 years ago

This seems to be #1480 but with the bonus that it totally breaks the tombstone flow making the tombstone broken at concept level

MTRNord commented 2 years ago

Just realised this is a dup of the issue I linked above. 😅