nullsoepic / liveunderflow

https://discord.gg/F9qCKC8TCk ~ Discord bot for a LiveOverflow community
Creative Commons Attribution Share Alike 4.0 International
5 stars 4 forks source link

Message filter deletes IP in application Tickets #11

Closed ATXLtheAxolotl closed 1 year ago

ATXLtheAxolotl commented 1 year ago

Now normally we would not want people sharing the ip. However when applicants are requested the ip the bot deletes it. It does say “message contains blacklisted phrase” but it would be clearer if tickets were exempt from this blacklist.

XeynQ4 commented 1 year ago

i think the bot should replace the messages containing the right ips with a reply saying "you sent the correct server/proxy ip" or something like that. that way we can verify if he sent the correct one and there will be no leaks. this could be in all channels, not just the application one

ATXLtheAxolotl commented 1 year ago

i think the bot should replace the messages containing the right ips with a reply saying "you sent the correct server/proxy ip" or something like that. that way we can verify if he sent the correct one and there will be no leaks. this could be in all channels, not just the application one

At that point I feel like it may be a repeat of /check, maybe we could go with one of my previous ideas where if they run /check and get it right it will so say in their tickets.

veronoicc commented 1 year ago

Id say just exempting the blacklist from tickets would fine, since the application can only be seen by people that know the ip already anyway

veronoicc commented 1 year ago

The code for removing the ip can be deprecated, as Discord' AutoMod already does that job (and excludes Tickets!)

veronoicc commented 1 year ago

I did not mean to reopen sorry