Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Support ignoring users or hostmasks #42

Open
j0rdan76 opened this issue May 12, 2014 · 6 comments
Open

Support ignoring users or hostmasks #42

j0rdan76 opened this issue May 12, 2014 · 6 comments

Comments

@j0rdan76
Copy link

Have a way to ignore hostmasks or users would be very beneficial to help specific users that keep spamming Ultros.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@gdude2002 gdude2002 self-assigned this May 12, 2014
@gdude2002 gdude2002 added this to the 1.1 milestone May 12, 2014
@gdude2002
Copy link
Member

This is going to require some protocol extensions; I'll look into it.

@rakiru
Copy link
Member

rakiru commented May 12, 2014

I was planning this for a plugin. It should be doable that way, with the ability to either set an explicit mask, or ignore a user and have the plugin create a mask based on it. It can just cancel incoming message events from ignored users or something.

@gdude2002
Copy link
Member

Well the point I'm making is that if we're going to do /proper/ ignores then the protocols should have a function that can match users based on strings.

@rakiru
Copy link
Member

rakiru commented May 12, 2014

Ah right. Well, the IRC protocol has a utils thing that has suitable methods, and for mumble it's pretty much just nick comparison.
I had planned to before make protocol_inst.utils a thing on every protocol, but never got around to figuring out the best way to go about it. Having a base utils class with required functionality, then having protocols subclass that would probably be the best way. At that point, we might be better off just putting the functions in the protocol class itself, but the protocols are already pretty massive.

@gdude2002
Copy link
Member

Well.. The base protocol classes are pretty small, the /actual/ protocols are large. But this definitely would be protocol-specific.

@gdude2002
Copy link
Member

Blocked by #51

@gdude2002 gdude2002 removed this from the 1.1 milestone Aug 20, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants