View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003993 | unreal | ircd | public | 2010-12-31 04:27 | 2016-02-29 14:28 |
Reporter | Aubrey | Assigned To | syzop | ||
Priority | normal | Severity | feature | Reproducibility | N/A |
Status | resolved | Resolution | fixed | ||
Product Version | 3.2.9-RC1 | ||||
Fixed in Version | 4.0.2 | ||||
Summary | 0003993: Add ability to hide quit messages of glined users | ||||
Description | When a user is banned from the server in any way, their quit message is "User has been banned (?line reason here)". A feature to hide the ban reason from everyone or even only non-opers would be very useful. Frequently ban messages are private and don't need to be shared with the world. | ||||
Tags | No tags attached. | ||||
3rd party modules | |||||
|
I see the feature being abused and don't really think there is any reason to keep the reason hidden. The reason at times can help other users (friend of the user banned) to approach the operators and clarify the issue. |
|
Server bans are (in most IRC environments) not a public issue. It also encourages opers to use glines as a "warning message" to others which isn't appropriate. A setting to turn this on or off à-la InspIRCd would be nice. |
|
I'm fine with this. Not so sure if I'll spend time on it, though. |
|
Apparently there's a pull request and (thus) patch available https://github.com/Zoddo/unrealircd/commit/57f01ea3b9796bfb69a9cf973f87fab2bad19ef0 Don't like set::options::secret-ban-reasons, though (the name and to lesser extent place). Haven't walked through the patch line by line but the structure/solution looks good. Not sure if it's for 4.0.1 though, want to release that soon. |
|
secret sounds stupid 'hide-ban-reason' does the job ;-) also I would vote against the long 'User has been banned blah blah blah ...' string other IRC daemons just do 'X-Lined' or similar, looks much neater that's opinion though ;-) (either way, can someone make a decision so that we can get this for the next release? would be real nice) |
|
[unreal40 1e25578] Add ability to hide quit messages of *LINEd users. Suggested by Aubrey, patch mostly from Zoddo (0003993). https://github.com/unrealircd/unrealircd/commit/1e25578c42b6e68b87fd6b1eb73d87677c9052f9 I used the patch as a base (thanks!) and made it set::hide-ban-reason [yes|no] (the default is no). I'll update the wiki as well. The zline case (already) showed the reason if user was not really online yet, I updated the other *LINE cases as well. This may help to people who repeatedly try to connect and somehow miss the server notice. I did not integrate the example.conf stuff |
Date Modified | Username | Field | Change |
---|---|---|---|
2010-12-31 04:27 | Aubrey | New Issue | |
2010-12-31 08:28 | goldenwolf | Note Added: 0016549 | |
2010-12-31 15:57 | Aubrey | Note Added: 0016555 | |
2015-07-13 22:25 | syzop | Note Added: 0018492 | |
2015-07-13 22:25 | syzop | Status | new => acknowledged |
2016-01-13 11:14 | syzop | Note Added: 0019057 | |
2016-01-13 11:14 | syzop | Status | acknowledged => has patch |
2016-01-13 11:15 | syzop | Note Edited: 0019057 | |
2016-01-13 11:15 | syzop | Note Edited: 0019057 | |
2016-01-13 11:58 | blank | Note Added: 0019059 | |
2016-01-13 12:02 | blank | Note Edited: 0019059 | |
2016-02-29 14:28 | syzop | Note Added: 0019112 | |
2016-02-29 14:28 | syzop | Status | has patch => resolved |
2016-02-29 14:28 | syzop | Fixed in Version | => 4.0.2 |
2016-02-29 14:28 | syzop | Resolution | open => fixed |
2016-02-29 14:28 | syzop | Assigned To | => syzop |