GriefPrevention / GriefPrevention

The official self-service anti-griefing Bukkit plugin for Minecraft servers since 2011.
https://griefprevention.com
GNU General Public License v3.0
296 stars 240 forks source link

Players able to break stuff in claims of admins and above #2296

Closed wojcake closed 4 months ago

wojcake commented 4 months ago

Observed Behavior

Basically, members can break claims made by people with admin rank and higher, not lower. They can't break other member's claims nor moderators'

Expected Behavior

They should not be able to break stuff in any claims they are not trusted in

Reproduction steps

  1. Admin claims land
  2. Member comes to the claim
  3. Breaks block even though they have no /trust
  4. Member claims land
  5. Other member comes to the claim
  6. Can't break anything

Stack trace or error log

No response

Server version

[18:01:04 INFO]: This server is running Paper version git-Paper-496 (MC: 1.20.4) (Implementing API version 1.20.4-R0.1-SNAPSHOT) (Git: 7ac24a1 on ver/1.20.4)
You are running the latest version
Previous version: git-Paper-424 (MC: 1.20.4)

GriefPrevention version

GriefPrevention version 16.18.2

Configuration

# Default values are perfect for most servers.  If you want to customize and have a question, look for the answer here first: http://dev.bukkit.org/bukkit-plugins/grief-prevention/pages/setup-and-configuration/

GriefPrevention:
  SeaLevelOverrides:
    specturecraft: -1
    specturecraft_nether: -1
    specturecraft_the_end: -1
  Claims:
    Mode:
      specturecraft_nether: Survival
      specturecraft_the_end: Survival
      specturecraft: Survival
    PreventGlobalMonsterEggs: true
    PreventTheft: true
    ProtectCreatures: true
    PreventButtonsSwitches: false
    LockWoodenDoors: false
    LockTrapDoors: true
    LockFenceGates: true
    EnderPearlsRequireAccessTrust: true
    RaidTriggersRequireBuildTrust: true
    ProtectHorses: true
    ProtectDonkeys: true
    ProtectLlamas: true
    InitialBlocks: 50
    Claim Blocks Accrued Per Hour:
      Default: 200
    Max Accrued Claim Blocks:
      Default: 15000
    Accrued Idle Threshold: 0
    AccruedIdlePercent: 0
    AbandonReturnRatio: 1.0
    AutomaticNewPlayerClaimsRadius: 4
    AutomaticNewPlayerClaimsRadiusMinimum: 0
    ExtendIntoGroundDistance: 1000
    MinimumWidth: 4
    MinimumArea: 25
    MaximumDepth: -64
    InvestigationTool: STICK
    ModificationTool: GOLDEN_SHOVEL
    Expiration:
      ChestClaimDays: 7
      UnusedClaimDays: 30
      AllClaims:
        DaysInactive: 30
        ExceptWhenOwnerHasTotalClaimBlocks: 2000
        ExceptWhenOwnerHasBonusClaimBlocks: 5000
      AutomaticNatureRestoration:
        SurvivalWorlds: false
    AllowTrappedInAdminClaims: true
    MaximumNumberOfClaimsPerPlayer: 0
    CreationRequiresWorldGuardBuildPermission: true
    VillagerTradingRequiresPermission: true
    CommandsRequiringAccessTrust: /sethome
    DeliverManuals: true
    ManualDeliveryDelaySeconds: 30
    RavagersBreakBlocks: true
    FireSpreadsInClaims: false
    FireDamagesInClaims: false
    LecternReadingRequiresAccessTrust: true
  Spam:
    Enabled: false
    LoginCooldownSeconds: 60
    LoginLogoutNotificationsPerMinute: 5
    ChatSlashCommands: /me;/global;/local
    WhisperSlashCommands: /tell;/pm;/r;/whisper;/msg;/m;/r
    WarningMessage: Please reduce your noise level.  Spammers will be banned.
    BanOffenders: false
    BanMessage: Banned for spam.
    AllowedIpAddresses: 1.2.3.4; 5.6.7.8
    DeathMessageCooldownSeconds: 120
    Logout Message Delay In Seconds: 0
  PvP:
    RulesEnabledInWorld:
      specturecraft: true
      specturecraft_nether: true
      specturecraft_the_end: true
    ProtectFreshSpawns: true
    PunishLogout: true
    CombatTimeoutSeconds: 10
    AllowCombatItemDrop: false
    BlockedSlashCommands: /home;/vanish;/spawn;/tpa
    ProtectPlayersInLandClaims:
      PlayerOwnedClaims: false
      AdministrativeClaims: false
      AdministrativeSubdivisions: false
    AllowLavaDumpingNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: true
    AllowFlintAndSteelNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: false
    ProtectPetsOutsideLandClaims: true
  Economy:
    ClaimBlocksMaxBonus: 0
    ClaimBlocksPurchaseCost: 0.0
    ClaimBlocksSellValue: 0.0
  ProtectItemsDroppedOnDeath:
    PvPWorlds: false
    NonPvPWorlds: true
  BlockLandClaimExplosions: true
  BlockSurfaceCreeperExplosions: false
  BlockSurfaceOtherExplosions: false
  LimitSkyTrees: true
  LimitTreeGrowth: false
  PistonMovement: CLAIMS_ONLY
  PistonExplosionSound: true
  FireSpreads: false
  FireDestroys: false
  AdminsGetWhispers: true
  AdminsGetSignNotifications: true
  VisualizationAntiCheatCompatMode: false
  SmartBan: false
  Mute New Players Using Banned Words: false
  MaxPlayersPerIpAddress: 5
  SilenceBans: true
  Siege:
    Worlds: []
    BreakableBlocks:
    - GRASS_BLOCK
    - DIRT
    - COBBLESTONE
    - OAK_PLANKS
    - SPRUCE_PLANKS
    - BIRCH_PLANKS
    - JUNGLE_PLANKS
    - ACACIA_PLANKS
    - DARK_OAK_PLANKS
    - SAND
    - GRAVEL
    - GLASS
    - FERN
    - DEAD_BUSH
    - WHITE_WOOL
    - ORANGE_WOOL
    - MAGENTA_WOOL
    - LIGHT_BLUE_WOOL
    - YELLOW_WOOL
    - LIME_WOOL
    - PINK_WOOL
    - GRAY_WOOL
    - LIGHT_GRAY_WOOL
    - CYAN_WOOL
    - PURPLE_WOOL
    - BLUE_WOOL
    - BROWN_WOOL
    - GREEN_WOOL
    - RED_WOOL
    - BLACK_WOOL
    - SNOW
    - GLASS_PANE
    DoorsOpenDelayInSeconds: 300
    CooldownEndInMinutes: 60
  EndermenMoveBlocks: false
  SilverfishBreakBlocks: false
  CreaturesTrampleCrops: false
  RabbitsEatCrops: true
  HardModeZombiesBreakDoors: false
  MobProjectilesChangeBlocks: false
  Database:
    URL: ''
    UserName: ''
    Password: ''
  UseBanCommand: false
  BanCommandPattern: ban %name% %reason%
  Advanced:
    fixNegativeClaimblockAmounts: true
    ClaimExpirationCheckRate: 30
    OfflinePlayer_cache_days: 50
  Abridged Logs:
    Days To Keep: 7
    Included Entry Types:
      Social Activity: true
      Suspicious Activity: true
      Administrative Activity: false
      Debug: false
      Muted Chat Messages: false
  ConfigVersion: 1

Plugin list

Paper Plugins:
[18:02:17 INFO]:  - MarriageMaster, Minepacks
[18:02:17 INFO]: Bukkit Plugins:
[18:02:17 INFO]:  - AuthMe, Autorank, BanFromClaim, BetterRTP, ChatManager, CMILib, CMILib, CMILib, CMILib, CMILib
[18:02:17 INFO]:  CMILib, CMILib, CommandPanels, CoreProtect, CrystamaeHistoria, CustomAdvancements, DecentHolograms, DiscordSRV, Essentials, EssentialsChat
[18:02:17 INFO]:  EssentialsSpawn, ExcellentCrates, FarmingUpgrade, floodgate, FlowerPower, FluffyMachines, FoxyMachines, Geyser-Spigot, GriefPrevention, GSit
[18:02:17 INFO]:  Harbor, ImageMaps, InfinityExpansion, InventoryRollbackPlus, Jobs, LiteBans, LuckPerms, mcMMO, Networks, NexEngine
[18:02:17 INFO]:  OpenInv, PlaceholderAPI, PluginManager, PomaExpansion, ProtocolLib, PvPManager, QuickShop-Hikari, RelicsOfCthonia, SFCalc, Shopkeepers
[18:02:17 INFO]:  SimpleClans, SimpleStorage, SkinsRestorer, Slimefun, SlimeTinker, spark, SuperVanish, TAB, Terra, Themis
[18:02:17 INFO]:  TranscEndence, UltimateAdvancementAPI, UltimateAutoRestart, Vault, VeinMiner, ViaBackwards, ViaRewind, ViaVersion, Votifier, VotingPlugin
[18:02:17 INFO]:  WorldEdit, WorldGuard, WorldGuardExtraFlags

Running without GriefPrevention

Running with only GriefPrevention

Running on a fresh, clean server installation

Using unmodified client

We appreciate you taking the time to fill out a bug report!

Jikoo commented 4 months ago

Sounds like a plugin conflict, can you replicate with only GP installed?

wojcake commented 4 months ago

Solved, config in SimpleClans was allowing to break stuff in war