Grief Prevention

Grief Prevention

Bukkit Plugins
1M Downloads

Boats can be leashed inside claimed land and ridden by players without trust.

IndrajeethY opened this issue · 3 comments

commented

Observed Behavior

  • Untrusted players can attach leads to boats (and possibly minecarts) inside claimed land.
  • They can pull these boats into and around claimed areas.
  • They can also ride the boats inside claims without having any trust permissions.

Expected Behavior

  • Untrusted players should not be able to leash, move, or ride boats/minecarts that are located inside a claim without proper trust permissions.

Reproduction steps

  1. Place a boat inside a claim.
  2. Have another player with no trust in the claim attempt to attach a lead to the boat.
  3. Observe that they can pull the boat with leads and ride it inside the claim without restrictions.

Stack trace or error log

* N/A (behavioral bug, no console errors).

Server version

This server is running Paper version 1.21.4-232-ver/1.21.4@12d8fe0 (2025-06-09T10:15:42Z) (Implementing API version 1.21.4-R0.1-SNAPSHOT)
You are running the latest version

GriefPrevention version

16.18.4

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:
    world: -1
    world_nether: -1
    world_the_end: -1
  Claims:
    Mode:
      world_nether: Disabled
      world_the_end: Disabled
      world: Survival
    PreventGlobalMonsterEggs: true
    PreventTheft: true
    ProtectCreatures: true
    PreventButtonsSwitches: true
    LockWoodenDoors: false
    LockTrapDoors: false
    LockFenceGates: true
    EnderPearlsRequireAccessTrust: true
    RaidTriggersRequireBuildTrust: true
    ProtectHorses: true
    ProtectDonkeys: true
    ProtectLlamas: true
    InitialBlocks: 100
    Claim Blocks Accrued Per Hour:
      Default: 100
    Max Accrued Claim Blocks:
      Default: 80000
    Accrued Idle Threshold: 0
    AccruedIdlePercent: 0
    AbandonReturnRatio: 1.0
    AutomaticNewPlayerClaimsRadius: 4
    AutomaticNewPlayerClaimsRadiusMinimum: 0
    ExtendIntoGroundDistance: 5
    MinimumWidth: 5
    MinimumArea: 100
    MaximumDepth: -2147483648
    InvestigationTool: STICK
    ModificationTool: GOLDEN_SHOVEL
    Expiration:
      ChestClaimDays: 7
      UnusedClaimDays: 14
      AllClaims:
        DaysInactive: 60
        ExceptWhenOwnerHasTotalClaimBlocks: 10000
        ExceptWhenOwnerHasBonusClaimBlocks: 5000
      AutomaticNatureRestoration:
        SurvivalWorlds: false
    AllowTrappedInAdminClaims: false
    MaximumNumberOfClaimsPerPlayer: 0
    CreationRequiresWorldGuardBuildPermission: true
    VillagerTradingRequiresPermission: true
    CommandsRequiringAccessTrust: /sethome
    DeliverManuals: true
    ManualDeliveryDelaySeconds: 30
    RavagersBreakBlocks: true
    FireSpreadsInClaims: false
    FireDamagesInClaims: false
    LecternReadingRequiresAccessTrust: true
  Spam:
    Enabled: true
    LoginCooldownSeconds: 60
    LoginLogoutNotificationsPerMinute: 5
    ChatSlashCommands: /me;/global;/local
    WhisperSlashCommands: /tell;/pm;/r;/whisper;/msg
    WarningMessage: Please reduce your noise level.  Spammers will be banned.
    BanOffenders: true
    BanMessage: Banned for spam.
    AllowedIpAddresses: 1.2.3.4; 5.6.7.8
    DeathMessageCooldownSeconds: 120
    Logout Message Delay In Seconds: 0
  PvP:
    RulesEnabledInWorld:
      world: true
      world_nether: true
      world_the_end: true
    ProtectFreshSpawns: true
    PunishLogout: true
    CombatTimeoutSeconds: 15
    AllowCombatItemDrop: false
    BlockedSlashCommands: /home;/vanish;/spawn;/tpa
    ProtectPlayersInLandClaims:
      PlayerOwnedClaims: true
      AdministrativeClaims: true
      AdministrativeSubdivisions: true
    AllowLavaDumpingNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: false
    AllowFlintAndSteelNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: false
    ProtectPetsOutsideLandClaims: false
  Economy:
    ClaimBlocksMaxBonus: 0
    ClaimBlocksPurchaseCost: 0.0
    ClaimBlocksSellValue: 0.0
  ProtectItemsDroppedOnDeath:
    PvPWorlds: false
    NonPvPWorlds: true
  BlockLandClaimExplosions: true
  BlockSurfaceCreeperExplosions: true
  BlockSurfaceOtherExplosions: true
  LimitSkyTrees: true
  LimitTreeGrowth: false
  PistonMovement: CLAIMS_ONLY
  PistonExplosionSound: true
  FireSpreads: false
  FireDestroys: false
  AdminsGetWhispers: true
  AdminsGetSignNotifications: true
  VisualizationAntiCheatCompatMode: false
  SmartBan: true
  Mute New Players Using Banned Words: true
  MaxPlayersPerIpAddress: 3
  SilenceBans: true
  Siege:
    Worlds: []
    BreakableBlocks:
    - DARK_OAK_PLANKS
    - PURPLE_WOOL
    - GLASS
    - ORANGE_WOOL
    - COBBLESTONE
    - OAK_PLANKS
    - ACACIA_PLANKS
    - BROWN_WOOL
    - LIGHT_GRAY_WOOL
    - JUNGLE_PLANKS
    - RED_WOOL
    - GREEN_WOOL
    - BIRCH_PLANKS
    - SAND
    - GRAY_WOOL
    - GRASS_BLOCK
    - YELLOW_WOOL
    - PINK_WOOL
    - LIME_WOOL
    - SHORT_GRASS
    - GLASS_PANE
    - FERN
    - BLACK_WOOL
    - CYAN_WOOL
    - DEAD_BUSH
    - SNOW
    - BLUE_WOOL
    - GRAVEL
    - WHITE_WOOL
    - LIGHT_BLUE_WOOL
    - MAGENTA_WOOL
    - SPRUCE_PLANKS
    - DIRT
    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: 60
    OfflinePlayer_cache_days: 90
  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

[05:36:55 INFO]: ℹ Server Plugins (13):
[05:36:55 INFO]: Paper Plugins (2):
[05:36:55 INFO]:  - ExcellentEnchants, nightcore
[05:36:55 INFO]: Bukkit Plugins (11):
[05:36:55 INFO]:  - AuraSkills, AuthMe, CoreProtect, Essentials, GriefPrevention, ProtocolLib, SmartSpawner, SmartSpawner, ViaBackwards, ViaVersion
[05:36:55 INFO]:  WorldEdit
commented

If you're not going to provide plugin or configuration list, then you will need to affirm the testing that you deleted.

commented

If you're not going to provide plugin or configuration list, then you will need to affirm the testing that you deleted.

Aight, I've updated my issue with more info.

commented

Yea, will need to test without offline mode and protocol plugins, to ensure that this is indeed GP's fault. I believe current behavior is players aren't allowed to board boats and other vehicles in claims that are not theirs (though this causes a different issue that #295 and the like wish to solve)

Being able to tether vehicles - not sure how to best address this. When was this feature added btw?