View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006319 | unreal | ircd | public | 2023-07-26 10:40 | 2024-04-07 08:02 |
| Reporter | BlackBishop | Assigned To | |||
| Priority | normal | Severity | feature | Reproducibility | always |
| Status | acknowledged | Resolution | open | ||
| Summary | 0006319: "ban user" and "require authentication" is not a mask item | ||||
| Description | Trying to `ban user` seems to crash ban user { mask ~security-group: badpeople; reason "you shouldn't be here'; } Trying in another way security-group badpeople { mask { x.x.x.x/y; }; }; ban user { mask { security-group { badpeople; }; }; reason "You shouldn't be here!"; }; Results in: ban::mask specified without a value ban::mask is missing | ||||
| Tags | No tags attached. | ||||
| 3rd party modules | |||||
|
|
The example `Ban all except` in https://www.unrealircd.org/docs/Ban_user_block doesn't work either. I just can't make ban user work for some reason. And `mask ~security-group: badpeople;` is a reproducible crash on another platform also. I know the syntax is wrong, but it still shouldn't crash. |
|
|
Fixed the crash:commit 2665cec73bad67acc79232e633c8887352525c97 (HEAD -> unreal60_dev, origin/unreal60_dev, origin/HEAD) Author: Bram Matthys <[email protected]> Date: Wed Jul 26 12:44:31 2023 +0200 Fix crash when ~security-group:securitygroup is used in conf (so old style in eg ban user::mask). Reported by BlackBishop in https://bugs.unrealircd.org/view.php?id=6319 But you are right, the 'new' syntax / mask item syntax (and thus the example) does not work in "ban user". The code uses the old style. While many other places including "except ban" support the new style (Mask item). I will leave this bug open until that issue is resolved, I won't have time for this in the next few weeks. |
|
|
Thank you! |
|
|
Renamed to "ban user is not a mask item" and is now a feature request. Documentation adjusted for now. |
|
|
Merged report 0006397 in here: this is now also about "require authentication", which nearly uses the same code. Both ban user { } and require authentication { } will support a real mask item somewhere in the future. Don't have an exact time or which release. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-07-26 10:40 | BlackBishop | New Issue | |
| 2023-07-26 11:17 | BlackBishop | Note Added: 0022980 | |
| 2023-07-26 12:49 | syzop | Note Added: 0022981 | |
| 2023-07-26 12:49 | syzop | Assigned To | => syzop |
| 2023-07-26 12:49 | syzop | Status | new => confirmed |
| 2023-07-26 12:49 | syzop | Target Version | => 6.1.2-rc1 |
| 2023-07-26 13:05 | BlackBishop | Note Added: 0022982 | |
| 2023-09-09 10:54 | syzop | Severity | minor => feature |
| 2023-09-09 10:54 | syzop | Target Version | 6.1.2-rc1 => |
| 2023-09-09 10:54 | syzop | Summary | Trying to ban user with security group results in crash => ban user is not a mask item |
| 2023-09-09 10:54 | syzop | Note Added: 0023022 | |
| 2023-09-09 10:55 | syzop | Assigned To | syzop => |
| 2023-09-09 10:55 | syzop | Status | confirmed => acknowledged |
| 2024-04-07 08:01 | syzop | Summary | ban user is not a mask item => "ban user" and "require authentication" is not a mask item |
| 2024-04-07 08:02 | syzop | Note Added: 0023165 |
