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

add a force-accept command, remove opers' silently overriding +g #10

Open
edk0 opened this issue Jul 4, 2016 · 3 comments
Open

add a force-accept command, remove opers' silently overriding +g #10

edk0 opened this issue Jul 4, 2016 · 3 comments

Comments

@edk0
Copy link
Contributor

edk0 commented Jul 4, 2016

it would be nice if we had to explicitly choose to override a user's wishes in this case.

elemental-ircd seems to have a /OACCEPT we could probably port; see https://github.com/Elemental-IRCd/elemental-ircd/blob/0e7f632f4dabbb2b2c13487c22d70eaf48f6c56e/extensions/m_oaccept.c and https://github.com/Elemental-IRCd/elemental-ircd/blob/0e7f632f4dabbb2b2c13487c22d70eaf48f6c56e/modules/core/m_message.c

@sdaugherty
Copy link

Adding entries to a users ACCEPT list is even worse behavior than overriding +g in my opinion, because you'd be taking up entries in their list, which are already limited.

It probably would be more useful for there to be an oper-only user mode for overruling +g, or an oper-only FORCEMSG / FORCENOTICE This avoids overly complicating a number of existing applications, such as monitor bots that check CTCP VERSION to help detect malicious client connections, and opers warning users of inappropriate behavior.

@edk0
Copy link
Contributor Author

edk0 commented Aug 1, 2016

fwiw, the /oaccept there uses a separate list

@sdaugherty
Copy link

sdaugherty commented Aug 1, 2016

Cool, at least it's not filling up their accept lists, but that still doesn't address the problems with monitor bots, etc.

Seems like it would be easier to just integrate the override into the PRIVMSG/NOTICE syntax somehow, that way it's a single command, no additional data structures have to be added, and the overrides can be readily incorporated into monitor bots and scripts.

Something like:
-> /msg USER :Hello (blocked)
<- * USER is in +g mode (server-side ignore) and has been informed you've messaged them
-> /msg =USER :Hello (message gets delivered to user)

@spb spb added the enhancement label Nov 3, 2017
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