Dotsimus sends triggered keyword message to users even if they arent in the server

[MetkarAdi]

No description provided.

[itsWindows11]

What? Lemme investigate

EDIT: do you mean that upon leaving the server you have a watched keyword(s) in, it doesn't remove the keywords watched?

[itsWindows11]

Upon further inspection in index.js, I found some code that's related to that but idk, that makes me suspect that this issue is invalid:

          const guild = client.guilds.cache.get(server.id);
          if (!guild.members.cache.get(watchedKeywordsGuild.userId)) {
            db.removeWatchedKeyword(watchedKeywordsGuild.userId, server.id).then(resp => {
              refreshWatchedCollection()
              console.info('Removed watcher: ' + watchedKeywordsGuild.userId)
              return;
            })
          }

[MetkarAdi]

I did have multiple users from another server complaining to me that "dot kept sending dms" even after they were banned.

[itsWindows11]

I did have multiple users from another server complaining to me that "dot kept sending dms" even after they were banned.

Hmm then lemme look into it

[Jasius]

It's kind of weird if this happens, as Dot checks whether user has access to a channel before sending a message, so if user isn't in the server then it cannot check for permissions and should auto remove them, will investigate this in the near future.

[UntrustedInstaller]

Agreed, shouldn't be doing that.

[itsWindows11]

Upon further inspection in index.js, I found some code that's related to that but idk, that makes me suspect that this issue is invalid:

          const guild = client.guilds.cache.get(server.id);
          if (!guild.members.cache.get(watchedKeywordsGuild.userId)) {
            db.removeWatchedKeyword(watchedKeywordsGuild.userId, server.id).then(resp => {
              refreshWatchedCollection()
              console.info('Removed watcher: ' + watchedKeywordsGuild.userId)
              return;
            })
          }

I think the issue might be it entirely depending on cache to get user/guild and check if user is in a guild, sometimes cache might be out of date.