Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

seen command giving uuid #4499

Closed
elaineandsparky opened this issue Aug 30, 2021 · 4 comments · Fixed by #4531
Closed

seen command giving uuid #4499

elaineandsparky opened this issue Aug 30, 2021 · 4 comments · Fixed by #4531
Labels
type: enhancement Features and feature requests.

Comments

@elaineandsparky
Copy link

Type of bug

Other unexpected behaviour

/ess version output

30.08 13:01:51 [Server] INFO Server version: 1.17.1-R0.1-SNAPSHOT git-Paper-176 (MC: 1.17.1)
30.08 13:01:51 [Server] INFO Brand version: Paper
30.08 13:01:51 [Server] INFO EssentialsX version: 2.19.0-dev+257-4bee159
30.08 13:01:51 [Server] INFO PlaceholderAPI version: 2.10.10
30.08 13:01:51 [Server] INFO LuckPerms version: 5.3.47
30.08 13:01:51 [Server] INFO Vault version: 1.7.3-b131
30.08 13:01:51 [Server] INFO EssentialsXChat version: 2.19.0-dev+257-4bee159
30.08 13:01:51 [Server] INFO EssentialsXProtect version: 2.19.0-dev+257-4bee159
30.08 13:01:51 [Server] INFO EssentialsXSpawn version: 2.19.0-dev+257-4bee159
30.08 13:01:51 [Server] INFO Citizens version: 2.0.28-SNAPSHOT (build 2357)
30.08 13:01:51 [Server] INFO DiscordSRV version: 1.23.0
30.08 13:01:51 [Server] INFO Economy Layer: None
30.08 13:01:51 [Server] INFO Fetching version information...
30.08 13:01:51 [Server] INFO You're running the latest EssentialsX dev build!

Server startup log

https://paste.gg/p/anonymous/dd2760b9fba74d47a6f6b83f8178f662

EssentialsX config files

https://paste.gg/p/anonymous/fa1651b9a9764152a160f131f8891309

Error log (if applicable)

No response

Bug description

players with just the essentials.seen permission can see the uuid of a player.

Note, these players only have the basic seen permission not essentials.seen.extra/ip/location

Steps to reproduce

give player essentials.seen permission
use the seen command as that player

Expected behaviour

for /seen to only return how long that player has been online or how long since they last logged on

Actual behaviour

gives the above information plus the players uuid

@elaineandsparky elaineandsparky added the bug: unconfirmed Potential bugs that need replicating to verify. label Aug 30, 2021
@Chew
Copy link
Member

Chew commented Aug 30, 2021

UUID isn't private information, you can easily get any user's UUID...

@molor
Copy link

molor commented Aug 30, 2021

UUID isn't private information, you can easily get any user's UUID...

But is it really necessary for regular players, many of whom do not even know what it is?..

@Chew
Copy link
Member

Chew commented Aug 31, 2021

Fair, this is more of a feature request, though (ability to hide the UUID)

@Chew Chew added type: enhancement Features and feature requests. and removed bug: unconfirmed Potential bugs that need replicating to verify. labels Aug 31, 2021
@elaineandsparky
Copy link
Author

If it is working as intended then what's the difference between seen and seen.extra please as I saw no difference when testing.
I do also agree it's not needed for normal players and doesn't look good to show it even if it is easily accessible elsewhere, which not everyone knows about.

mdcfe pushed a commit that referenced this issue Sep 28, 2021
Adds `essentials.seen.uuid` permission requirement to show uuids in `/seen`. People were complaining this cluttered it so whatever. Also adds said permission to the `essentials.seen.extra` permission group.

Closes #4499.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Features and feature requests.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants