EssentialsX

EssentialsX

2M Downloads

Sethome limit not enforced correctly

zefir-git opened this issue ยท 3 comments

commented

Type of bug

Other unexpected behaviour

/ess version output

[18:54:58 INFO]: Server version: 1.16.5-R0.1-SNAPSHOT git-Paper-777 (MC: 1.16.5)
[18:54:58 INFO]: EssentialsX version: 2.18.2.0
[18:54:58 INFO]: PlaceholderAPI version: 2.10.9
[18:54:58 INFO]: LuckPerms version: 5.3.47
[18:54:58 INFO]: Vault version: 1.7.3-b131
[18:54:58 INFO]: EssentialsXSpawn version: 2.18.2.0
[18:54:58 INFO]: ChestShop version: 3.12 (build 307)
[18:54:58 INFO]: You are running an unsupported server version!

Server startup log

https://paste.gg/p/anonymous/5bbb1e01143740fabba52a3338265c2d

EssentialsX config files

https://paste.gg/p/anonymous/19a9a5b70e6547c5b4b9e6e9356f4ac3

Error log (if applicable)

No response

Bug description

A player is allowed to set an arbitrary number of homes (nowhere defined).
image
image
Upon /sethome d
image
Permission:

image

Steps to reproduce

  1. Use reported software versions
  2. Copy the config section shared
  3. Give permissions
  • essentials.sethome
  • essentials.sethome.multiple
  • essentials.sethome.multiple.tier1
  1. Run /sethome a, /sethome b, /sethome c until error is shown.

Expected behaviour

The player should only be able to set 2 homes

Actual behaviour

The player is able to set 3 homes. A number nowhere used in the configuration

commented

This is because Essentials tries to find the greatest number of homes that a user can have, and because the default default sethome limit is 3. Any user with the permission essentials.sethome.multiple is automatically considered for the default limit.

You can set the default key to 0 or something less than any other key under sethome-multiple to have your desired behaviour.

commented

This is because Essentials tries to find the greatest number of homes that a user can have, and because the default default sethome limit is 3. Any user with the permission essentials.sethome.multiple is automatically considered for the default limit.

You can set the default key to 0 or something less than any other key under sethome-multiple to have your desired behaviour.

How is the default 3? Where do I change that? My default group does not have essentials.sethome.multiple

commented

Essentials uses the sethome-multiple.default key to determine what the default home limit is. If the key is not present in the config, Essentials uses 3 as the value as a fallback. You can manually override it by explicitly setting a default key in your config, like this:

sethome-multiple:
  default: 1
  tier1: 2