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

Why does unban have apply, and why is it default-false? #476

Open
grahamc opened this issue Feb 7, 2023 · 1 comment
Open

Why does unban have apply, and why is it default-false? #476

grahamc opened this issue Feb 7, 2023 · 1 comment

Comments

@grahamc
Copy link
Contributor

grahamc commented Feb 7, 2023

I find I almost always unban first without apply. Then, when I realize my mistake I have to re-ban them then unban them again with apply for it to actually work.

... why is unban built the way that it is?

@turt2live
Copy link
Member

Originally Mjolnir was intended to be purely a way to manage banlists, then during development it gained the ability to actually do the bans itself. The unban command was thought to be more dangerous, so a flag was added to actually apply the unban.

The other reason is internally to the code is no logical separation between an unban issued by a list you control versus a list you don't control: for safety, unbans are not automatically applied and instead someone needs to manually apply them with the unban command.

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

No branches or pull requests

2 participants