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

add config option to use klinechan reasons as k-line reasons #54

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jesopo
Copy link
Member

@jesopo jesopo commented Dec 3, 2021

No description provided.

@maxteufel
Copy link
Member

Perhaps the config option could be named just klinechan_public, and if enabled also expose the fact that a channel is a klinechan and the reason (without person who set it and timestamp) in ChanServ INFO output?

@jesopo
Copy link
Member Author

jesopo commented Dec 3, 2021

no, I don't want it to be public that these channels are klinechans, I just want to be able to use the reasons for k-line reasons.

usecase is that currently the k-line reason is Joining #foo whereas I'd like it to be Please do not spam Foobar Networks. Email bans@foobar for help.

this becomes extra helpful when you've got inlined oper-parts in k-line reasons like solanum does, and when you've got bots that read those oper reasons to be told to do things - Please do not spam Foobar Networks. Email bans@foobar for help.|spam bots used to flood #foo !dnsbl

@maxteufel
Copy link
Member

Fair enough. The question has come up a few times because it's quite easy to test whether a channel is a klinechan anyway.

Your usecase could also be implemented by changing the reason to include i.e. serverinfo::adminemail. It could also include a machine-readable oper reason like

Please do not spam serverinfo::netname. Email serverinfo::adminemail for help|KLINECHAN #channel

Then the klinechan reasons could be used for more specific (private) information, like whether a channel is used as a command-and-control channel for a botnet or whatever.

@jesopo
Copy link
Member Author

jesopo commented Dec 3, 2021

you can already do that for solanum because it supports inlined oper reasons. I guess the code could enforce this by truncating at |, that way it'd work on other IRCds too

@jesopo jesopo marked this pull request as draft December 3, 2021 14:15
@jesopo
Copy link
Member Author

jesopo commented Dec 3, 2021

converted to draft because I'm workshopping the idea of report-only honeypot channels, so external tooling can decide what network-specific thing to do when it sees a join to a bad channel

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