minetest-mods / inspector

A minetest node inspector tool that shows various properties of nodes.
GNU Lesser General Public License v2.1
9 stars 8 forks source link

[Feature request] pretty-print the node's meta, hide private meta #3

Open VanessaE opened 6 years ago

VanessaE commented 6 years ago

This should be addressed if possible:

image

The meta should be displayed in some more readable way, maybe just "dumped" similar to the node definition. Might be a good idea to do the same for the meta inventory, (though the node being inspected in that image is not affected).

Also, I think it would be a good idea to prevent the inspector from revealing "private" meta unless the wielder has sufficient privileges, or is the node's owner.

sofar commented 5 years ago

This item shouldn't be available by default, let's address that issue instead?

VanessaE commented 5 years ago

It's good that it is available though -- but perhaps one ought to have "server" privilege for that to show up?

FaceDeer commented 4 years ago

If my latest pull request goes through then only players with the "server" privilege will be able to use the inspector tool. That's already the case for the chat command version. Hopefully this will protect private metadata sufficiently while still letting modders see it for their work.