search

minetest-mods / xdecor

A decoration mod for Minetest meant to be light, simple and well-featured
Other
29 stars 46 forks source link

Logging for mailboxes, storage nodes #101

Closed ptvirgo closed 5 years ago

ptvirgo commented 5 years ago

Requested by aerozoic, admin for UGX realms: storage nodes and mailboxes log inventory actions (adding, removing) as appropriate.

kilbith commented 5 years ago

Why?

ptvirgo commented 5 years ago

It's a common problem in minetest that someone can have a locked house and want to use shelves for storage, but another player will use glitches / bugs to break into the house and steal things.

Logging gives admins the option to enforce rules against theft without changing how the storage nodes work from the player's perspective, so people who are sharing on purpose can continue to do so.

I did the mailbox because you have an open issue (#91) requesting it.

kilbith commented 5 years ago

Stealing is perfectly OK in my philosophy.

ptvirgo commented 5 years ago

One of the nice things about logs is that admins can choose whether or not they want to respond to them, so on servers where this is considered hacking they can treat it as such, and those where it's acceptable won't be affected.

aerozoic commented 5 years ago

@kilbith I highly suggest you reconsider this issue. All storage nodes should log activity, not just the mailbox.

The question you should ask is "Why not?" It has absolutely nothing to do with whether you or i feel that "stealing" is ok or not. There simply is no reason reject this suggestion. Minetest game itself logs all such actions with storage nodes. So do it for consistency. Having this log will provide numerous benefits and doesn't not cause a single issue whatsoever. It will help with troubleshooting technical issues, security issues, and will most definitely help with investigating internal server issues.

:large_orange_diamond:

kilbith commented 5 years ago

I will consider that.