A248 / LibertyBans

The be-all, end-all of discipline.
https://ci.hahota.net:8443/job/LibertyBans/
GNU Affero General Public License v3.0
168 stars 42 forks source link

[Feature Request]: Option to hide IPs in auto-show but not /alts #259

Open fuzz5566 opened 7 months ago

fuzz5566 commented 7 months ago

I have confirmed that ...

Description

There should be an option to hide IPs when using auto-show. For example if an admin/op is screensharing their game, they probably don't want players IP's possibly showing in chat. However they still may want to use the /alts feature to check IPs when they are not screensharing.

An even better alternative would be to customize the layout for both. That way auto-show could be compact any only show player names and detection types while the /alts command could go more in depth with the IPs and time.

Note: koxsosen on discord said this, "Alternatively, I believe I have found a solution. In messages_en.yml, there is an option under victim-display, censor-ip-addresses. This makes it, so players without the libertybans.admin.viewips permission won't be able to see IPs.

So let's say you have the admin, and moderator group. Both moderators, and admins have access to the auto-show feature, but only admins can use the /alts command. So if you enable this feature, and only add libertybans.admin.viewips to the admin group, then mods won't be able to see IPs, but admins would see them both in auto-show, and the /alts command.

I understand that this is not a full solution, but for different groups, it works. - Thinking about it, the /alts command works very similar to the auto-show feature, so if we are talking about a single group, and if you give permission for both, only censoring the IPs at one wouldn't make sense."