|Anonymous | Login | Signup for a new account||2014-03-12 10:19 CET|
|Main | My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0004006||unreal||ircd||public||2011-02-18 15:05||2012-12-10 13:00|
|Priority||normal||Severity||feature||Reproducibility||have not tried|
|Target Version||Fixed in Version|
|Summary||0004006: enhance static part/quit|
|Description||Allow opers to see the real part/quit reason so network spam rules can still be enforced without non-opers being spammed.|
|Additional Information||I'll do the patch, just gimme a bit.|
|Tags||No tags attached.|
|3rd party modules|
|Attached Files||static.quit.non.regnick.patch [^] (687 bytes) 2011-03-07 00:17|
I got to looking at the source code and found that exit_client sends to channel and not individual users, which makes sense, so this will be more trouble that it's worth it seems.
Opers can always use snomasks +c and +F for (dis)connects which should display the real quit anyway.
uploaded do'able patch. This patch serves my personal needs but may be incomplete for Unreal3.2.
This blocks all quit messages from non-identified users, ie umode -r users except opers. For general use, you may want to add another config item. example: set::static-unreg-quits or however.
edited on: 2012-08-17 12:35
regarding original report.. you already found out the problem ;)
regarding your patch: yes I suppose that would be a good addition: a feature to make all unregged users quiting use the static quit.
If you would modify the patch to make this a configurable option, then I'll add it.
This might be unrelated but I have a suggestion:
Allow opers to enable (mode 1, mode 2) or disable static part/quit messages.
When feature is enabled to mode 1, all users' part/quit messages will be turned to static messages.
When feature is enabled to mode 2, only unregistered users' part/quit messages will be turned to static messages.
When feature is enabled to mode 1 or 2, opers with certain snomasks (as mentioned above) will part/quit with their real message.
And the featured is turned off when it is disabled.
dboyz, your "mode 1" and snomasks idea also already happens with +s +cF. Currently static quit is on|off and +s +cF snomask can always see real quits.
what about set::options::static-quit::all|unreg|opers ??
I like your idea but is it possible to allow opers to change the settings of static-quit?
For example, an IRCop can issue a command:
|static-quit is one of those things that shouldn't change often, if at all, so it really doesn't need to be changed on the fly.|
edited on: 2012-12-10 13:00
I'm looking back at this report after about 4 months.
Actually, I'm not referring to change the static quit message. Instead I am referring to the enabling/disabling part of static quit. I find it unreasonable to leave the static quit message feature enabled for an extended period of time due to the probability of people misusing it. (as in 0004149)
Other new suggestions I can think of are:
1) Add more flexibility in the config
-- Allow static quit reason to be used for static part reason. Also, allow separate reasons to be settable for both settings.
-- Allow static part option <no|yes|part message> to share the same option with static quit reason. Also, allow separate options to be settable for both settings.
-- Allow anti spam quit message time to share the same value with anti spam part message time. Also allow separate value to be settable for both settings.
2) Any settings/features mentioned above that are not available now, should be implemented.
3) Should any of the shared option of settings/features mentioned above exist, then an update is needed on the official documentation. (because it reflects unclear information; I am writing this note 100% based on the document.)
|2011-02-18 15:05||katsklaw||New Issue|
|2011-02-19 21:26||katsklaw||Note Added: 0016597|
|2011-03-07 00:17||katsklaw||File Added: static.quit.non.regnick.patch|
|2011-03-07 00:23||katsklaw||Note Added: 0016613|
|2011-03-07 00:23||katsklaw||Status||new => has patch|
|2011-03-27 23:40||mahdkoosh||File Added: unrealircd.conf|
|2011-03-27 23:53||mahdkoosh||Note Added: 0016629|
|2012-08-17 12:34||syzop||Note Added: 0017074|
|2012-08-17 12:34||syzop||Status||has patch => acknowledged|
|2012-08-17 12:35||syzop||Note Edited: 0017074||View Revisions|
|2012-08-19 10:10||dboyz||Note Added: 0017096|
|2012-08-19 18:30||syzop||File Deleted: unrealircd.conf|
|2012-08-22 01:50||katsklaw||Note Added: 0017097|
|2012-08-23 17:37||dboyz||Note Added: 0017105|
|2012-08-23 21:22||katsklaw||Note Added: 0017106|
|2012-12-10 13:00||dboyz||Note Added: 0017252|
|2012-12-10 13:00||dboyz||Note Edited: 0017252||View Revisions|
|Copyright © 2000 - 2014 MantisBT Team|