Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modified m_timedbans to behave more like setting mode +b manually. #435

Closed
wants to merge 1 commit into from

Conversation

ShutterQuick
Copy link
Contributor

It now accepts extbans. If one enters something not an extban, or valid banmask, it'll add "!@", making it a nick-ban instead of borking.

Fixes #377

It now accepts extbans. If one enters something not an extban, or valid banmask, it'll add "!*@*", making it a nick-ban instead of borking.
@HelixSpiral
Copy link
Contributor

You have to be careful adding stuff to the end of banmaks, awhile ago there was a glitch where you could set a ban that was @******{repeat until end of line}, then inspircd would add *! to the front of the ban, making the ban longer than the accepted line length, resulting in an unremovable ban.

I think you may have made it possible to do this again.

@ShutterQuick
Copy link
Contributor Author

Welp, you're right. This clearly wasn't thought through well enough by me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants