UnrealIRCd Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004006unrealircdpublic2011-02-18 15:052012-12-10 13:00
Reporterkatsklaw 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityhave not tried
StatusacknowledgedResolutionopen 
PlatformOSOS Version
Product Version3.2.9-RC1 
Target VersionFixed in Version 
Summary0004006: enhance static part/quit
DescriptionAllow opers to see the real part/quit reason so network spam rules can still be enforced without non-opers being spammed.
Additional InformationI'll do the patch, just gimme a bit.
TagsNo tags attached.
3rd party modules
Attached Filespatch file icon static.quit.non.regnick.patch [^] (687 bytes) 2011-03-07 00:17

- Relationships

-  Notes
(0016597)
katsklaw (reporter)
2011-02-19 21:26

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.
(0016613)
katsklaw (reporter)
2011-03-07 00:23

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.
(0016629)
mahdkoosh (reporter)
2011-03-27 23:53

/quit
(0017074)
syzop (administrator)
2012-08-17 12:34
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.

(0017096)
dboyz (reporter)
2012-08-19 10:10

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.
(0017097)
katsklaw (reporter)
2012-08-22 01:50

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 ??
(0017105)
dboyz (reporter)
2012-08-23 17:37

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 all
(0017106)
katsklaw (reporter)
2012-08-23 21:22

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.
(0017252)
dboyz (reporter)
2012-12-10 13:00
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.)


- Issue History
Date Modified Username Field Change
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
Powered by Mantis Bugtracker