#paper

1 messages ยท Page 26 of 1

rustic folioBOT
rustic folioBOT
#
notTamion
[PaperMC/Paper] Pull request review comment: #12806 Feat: Improve attack cooldown API

Currently the ticker can overflow naturally

thats a highly unlikely and edge case scenario. Sure the int can overflow after a staggering 3 years, but by that time you will have way worse issues with other integers which also increment by time. we can't base software on such insane scenarios

Should I just clamp the value to a minimum of zero for both the getter and setter?

just the setter:
Preconditions.checkArgument(ticks >= 0, "Ticks can't be negative");

rustic folioBOT
#
sudofox
[PaperMC/Paper] Issue opened: #12847 Autocrafters fail to distribute items correctly with hopper-amount > 1

Expected behavior

PaperSpigot 1.21.5-114-ver/1.21.5@a1b3058 (I know the new one just came out, but almost none of the plugins we use are updated yet. Ideally, this will be fixed in a 1.21.7 build.

ticks-per:
  hopper-transfer: 6
  hopper-check: 1
hopper-amount: 8

When using an autocrafter with hopper-amount: 8, items no longer distribute into separate unoccupied input slots as expected. Instead, all 8 items are inserted into a single slot per transfer tick. This breaks most vanilla autocrafter circuits.

Observed/Actual behavior

All 8 items go into the first compatible slot. This causes slot imbalances and prevents the autocrafter from crafting unless manually corrected. This makes it difficult to make a reliable circuit to condense iron bars into blocks.

Ideally, items fed into an autocrafter via hopper would bottleneck to one per tick to be a bit more consistent in terms of vanilla behavior.

Steps/models to reproduce

https://imgur.com/qW02nOx

Plugin and D...

rustic folioBOT
#
x9nico
[PaperMC/Paper] Issue opened: #12849 [1.21.7] Maven dependency paper-api detected as a Vulnerability : CVE-2025-48924

Expected behavior

It shouldn't be a vulnerability

Observed/Actual behavior

When I'm opening my pom.xml file with Maven on Intellij IDEA, it shows me that paper-api is known as a Vulnerability about CVE-2025-48924

<img width="2425" height="362" alt="Image" src="https://github.com/user-attachments/assets/24914f07-6785-4b02-868d-01e9fbdfabb3" />

Steps/models to reproduce

1- Create Maven project
2- Add paper-api to dependencies with the paper repo.
3- Look Maven warning

Plugin and Datapack List

N/A because it's in Maven (Paper-api 1.21.7)

Paper version

N/A because it's in Maven (Paper-api 1.21.7)

Other

No response

rustic folioBOT
rustic folioBOT
#
BenchmarkMode
[PaperMC/Paper] Issue opened: #12850 Craftworld equals() relies on identity

Expected behavior

typo in craftworld equals()

public boolean equals(Object obj) {
if (obj == null) {
return false;
} else if (this.getClass() != obj.getClass()) {
return false;
} else {
CraftWorld other = (CraftWorld)obj;
return this.getUID() == other.getUID(); // uuid == uuid2 ?????????
}
}

This isn't a problem 99% of the time but i bet if a world was unloaded then loaded again this would break, .equals should be checking if its the same world, not exact same memory location

Observed/Actual behavior

a

Steps/models to reproduce

a

Plugin and Datapack List

a

Paper version

a

Other

a

#
Owen1212055
[PaperMC/Paper] New branch created: fix/dfu-falldamage-issue
#
Owen1212055
[PaperMC/Paper] branch deleted: fix/dfu-falldamage-issue
rustic folioBOT
#
SqueakyCleanSqueen
[PaperMC/Paper] Issue opened: #12852 Paper 1.21.7-29 Maven Central used as a CDN

Expected behavior

I simply created the server with the newest version of Paper, threw in 4 plugins- ProtocolLib, Simple Voice Chat, Veinminer, and Veinminer Enchant- and expected it to work.

Observed/Actual behavior

This error popped up in the server setup, seemingly disabling all plugins:

[19:22:23 WARN]: [MavenLibraryResolver] Use of Maven Central as a CDN is against the Maven Central Terms of Service. Use MavenLibraryResolver.MAVEN_CENTRAL_DEFAULT_MIRROR instead. java.lang.RuntimeException: Plugin used Maven Central for library resolution at io.papermc.paper.plugin.loader.library.impl.MavenLibraryResolver.addRepository(MavenLibraryResolver.java:124) ~[paper-api-1.21.7-R0.1-SNAPSHOT.jar:?] at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:184) ~[?:?] at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197) ~[?:?] at java.base/java.util.Iterator.forEachRemaining(Iterator.java:133) ~[?...

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
TauCubed
[PaperMC/Paper] New comment on pull request: #12806 Feat: Improve attack cooldown API

What happens if you want to preserve the attack cooldown when changing someone's held item?

int ticks = player.getCooledAttackStrength(0F);
player.getInventory().setItemInMainHand(otherItemWithDifferentAttars);
player.setCooledAttackStrength(ticks, 0F);

The result would be undefined as the attack speed attributes could have changed.
It would requite that you do more calculations to achieve this, which would break if mojang ever changed the calculation of attack speed vs ticks. Whereas a simple getter/setter for the attack ticker would not.

This might be of utility though, so perhaps an additional method for setCooledAttackStrength?

rustic folioBOT
rustic folioBOT
#
BenchmarkMode
[PaperMC/Paper] New comment on issue: #12848 stinky

I'm not understanding your issue with using the supported method, getRespawnLocation(boolean).

If the issue is the double redirect in javadoc deprecation, yeah, that can be changed.

getPotentialBedLocation is just deprecated cause its a bad name. beds aren't the only source of respawns, and it doesn't always return a bed location in modern minecraft.

There are two almost identical duplicated code methods, I think its an issue

rustic folioBOT
#
hjk321
[PaperMC/Paper] Pull request opened: #12853 Add resetPose to Player class

Paper adds the capability of setting a "fixed" pose that is not updated every tick like normal. In certain situations, a plugin might have set a fixed pose and now wishes to reset the pose to a "natural" state. Sure, we could just use setPose with the current pose and fixed=false, but you'd have to wait until the next tick for the pose to update, and sometimes you need to get information from the new pose now.

This commit adds a new method for the Player class that clears the fixed pose state and immediately determines the correct pose based on the player's current state. Relevant player data, such as eye height, is immediately updated as well.

A practical example for why this is useful: In my plugin, I have a fixed swimming pose on the server-side, but obviously this doesn't carry over to the client. When the player fires a bow or launches a projectile, this would be useful to clear the fixed pose, compare the eye heights, and adjust the location of the new projectile accordingl...

rustic folioBOT
#
tal5
[PaperMC/Paper] Issue opened: #12854 A way to upgrade old NBT data by plugins

Is your feature request related to a problem?

Before the PDC API was a thing, a lot of plugins stored data (such as custom item IDs) directly in the item's NBT/now the custom_data component.
Nowadays the PDC API would be the way to do this, but any plugin that used to store data directly in NBT wouldn't ever be able to migrate from that, as old items could be laying around anywhere.

Describe the solution you'd like.

An "NMS API" (like the connection one for example) to let plugins register DFU fixes for their custom data, so that they are able to properly switch to modern options like PDC.

Alternatively if you wouldn't want to directly expose the DFU (and seeing as most plugins' use cases will likely be stuff like a string for custom item IDs), a registerMove("oldNbtKey", "newPdcKey") sorta setup could also work if possible (and maybe even be in UnsafeValues instead of NMS).

Describe alternatives you've considered.

Less ideal, but if the API had any way to get & set t...

rustic folioBOT
rustic folioBOT
#
jpenilla
[PaperMC/Paper] branch deleted: chore/gradle-fill-update
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
TPS-Nighthunter
[PaperMC/Paper] Issue opened: #12856 Portals don't link correctly.

Expected behavior

  • Linking nether portals in 3D (x,y,z) for a chunk loader array in 3x3 grid. Meaning 2 chunks on overworld between the chunk the portals are in.

  • Inside 128 block range -> Portals should link to the closest portal depending on coords.

  • For chunk loaders that means Portal A drops players and items in Portal A nether side. Portal B on B nether side and so on.

  • Tested and confirmed in vanilla.

Observed/Actual behavior

Portals do link, but will sometimes forget what portal they linked to and prioritize a different one that is not alinged to it.

Steps/models to reproduce

  1. Test wdl 1 in vanilla with player and also throw items (place a few blocks to catch a thrown item since portal hovers over lava).

  2. Check F3 + G for mentioned chunk spacing (2 chunks).

  3. Test red to red, vice versa. Blue to blue, vice versa. Red to nether blue, vice versa. Blue to nether red, vice versa.

  4. Works perfectly.

  5. Copy world to plain 1.21.7 #32 server and redo it se...

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
kennytv
[PaperMC/Paper] branch deleted: dev/1.21.8
#
kennytv
[PaperMC/Paper] New tag created: 1.21.7
rustic folioBOT
#
lynxplay
[PaperMC/Paper] New comment on issue: #12858 server crash

I am unsure how much we can do there.

Introducing a sync chunk load is what it is. It is already scheduled at blocking priority in the chunk system.
The only potential "solution" here would be to consider clipping into an unloaded chunk a miss.
That however obviously is diverging from normal vanilla behaviour and might have unexpected outcomes.

rustic folioBOT
rustic folioBOT
#
Privatech38
[PaperMC/Paper] Issue opened: #12859 Default particle speed (extra) should be set to 0 instead of 1

Expected behavior

Particle speed should default to 0, NMS code for the particle command, with only a name and position argument, is:

Commands.argument("pos", Vec3Argument.vec3())
    .executes(
        context1 -> sendParticles(
            context1.getSource(),
            ParticleArgument.getParticle(context1, "name"),
            Vec3Argument.getVec3(context1, "pos"),
            Vec3.ZERO,
            0.0F,
            0,
            false,
            context1.getSource().getServer().getPlayerList().getPlayers()
        )
    )

Observed/Actual behavior

Spawning a particle with the API defaults the extra (speed) field to 1. paper-server code:
https://github.com/PaperMC/Paper/blob/6fb36e34b6bfc8442b8dbeb61cb82ad9ed39013b/paper-server/src/main/java/org/bukkit/craftbukkit/CraftWorld.java#L2166-L2169
and
https://github.com/PaperMC/Paper/blob/6fb36e34b6bfc8442b8dbeb61cb82ad9ed39013b/paper-server/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java#L3003-L3006

T...

#
Xefoti
[PaperMC/Paper] Issue opened: #12860 "Formatting was not a valid color: ยงm" 1.21.6 Paper

Stack trace

https://mclo.gs/JpWsHI0 crush_server.txt
https://mclo.gs/6YwqQEj latest.log

Plugin and Datapack List

Plug-ins: AuraSkills, AuthMe, ChatItem, Chunky, CoreProtect, Crystal-TPA, eco, EcoEnchants, Essentials, fast-leaf-decay, libreforge, PlayerKits2, ProtocolLib, PyroFishingPro, PyroLib, SkinsRestorer, TabListPing, Vault, WildRTP, WorldBorder

DataPacks: There are 3 data pack(s) enabled: [vanilla (built-in)], [file/bukkit (world)], [paper (built-in)]

Actions to reproduce (if known)

No response

Paper version

This server is running Paper version 1.21.6-48-main@4d854e6 (2025-06-30T09:48:37Z) (Implementing API version 1.21.6-R0.1-SNAPSHOT)

Other

rng crashes. stable server work for 24> hours and suddenly crashes out for no reason

rustic folioBOT
rustic folioBOT
#
tal5
[PaperMC/Paper] New comment on issue: #12854 A way to upgrade old NBT data by plugins

One general big issue is that DFU only applies to upgrades of a data version controlled by mojang; i.e. you wouldn't be able to apply it reliably to existing worlds unless you managed to get them to install it before they updated their server

Yeah that's a downside sadly, but seeing as it's more-or-less the only solution I feel like people would be fine with adding it into their plugin and only making the actual code changes after an MC version change (unless I'm missing something and it doesn't work like that?)

#
Nu11ified
[PaperMC/Paper] New comment on issue: #12860 "Formatting was not a valid color: ยงm" 1.21.6 Paper

This stack trace shows that the crash originates from the Minecraft server's internal world saving mechanism (DimensionDataStorage.saveLevelData and MinecraftServer.tickServer). The error message Formatting was not a

valid color: ยงm; Formatting was not a valid color: ยงn; Formatting was not a valid color: ยงo; Formatting was not a valid color: ยงk; Formatting was not a valid color: ยงl

It tells us that when the server is trying to save some data, it encounters these specific formatting codes and treats them as invalid colors.

I will take a look to see where exactly in the code we can make it so it doesn't default to crashing.

#
mbax
[PaperMC/Paper] New comment on issue: #12854 A way to upgrade old NBT data by plugins

As a plugin dev, if I were in this situation, I imagine I would be doing the following:

  • My plugin already would be refusing to load on newer MC versions than it recognizes, because of NMS-poking being dangerous.
  • On a new major release, such as 1.22 for our current time of talking, I would use this API to ensure all chunks migrate cleanly.
  • I would then leave this migrator API usage present for at least several years before even considering removing it.

I only see this having downsides for devs who have code that recklessly assumes things behave the same across versions if the code still runs.

rustic folioBOT
#
Nu11ified
[PaperMC/Paper] Pull request opened: #12862 The issue "Formatting was not a valid color: ยงm"

This pull request introduces enhancements to color and formatting code validation in the Adventure library, specifically improving error handling and validation logic for distinguishing between valid colors and formatting codes. The changes aim to provide clearer error messages and better handling of edge cases.

Improvements to color and formatting code validation:

  • AdventureCodecs.java: Added logic to handle cases where a string starts with the 'ยง' character, distinguishing between formatting codes (e.g., 'ยงk', 'ยงl') and valid colors. If a formatting code is detected, an appropriate error message is returned instead of treating it as a color.

  • PaperAdventure.java: Enhanced the error message in the asAdventure method to include the invalid ChatFormatting value when a non-color formatting code is passed, improving the clarity of exceptions.occurs because:

  1. The TextColor.parseColor method in Minecraft is trying to parse formatting codes (like ยงm, ยงn, ยงo, ยงk, ยงl) as colo...
#
Nu11ified
[PaperMC/Paper] Pull request opened: #12863 The issue "Formatting was not a valid color: ยงm"

This pull request improves error handling and validation in the AdventureCodecs and PaperAdventure classes. The changes ensure better differentiation between formatting codes and valid colors, enhancing clarity and robustness in handling text color conversions.

Error Handling Improvements:

  • AdventureCodecs (paper-server/src/main/java/io/papermc/paper/adventure/AdventureCodecs.java): Added checks to identify and reject formatting codes (e.g., ยงk, ยงl, etc.) when they are mistakenly passed as colors. This ensures that only valid color codes or named colors are processed, with appropriate error messages for invalid inputs.

  • PaperAdventure (paper-server/src/main/java/io/papermc/paper/adventure/PaperAdventure.java): Enhanced the error message in the asAdventure method to include the specific formatting code that caused the exception, providing more context for debugging.

#
Nu11ified
[PaperMC/Paper] New comment on issue: #12860 "Formatting was not a valid color: ยงm" 1.21.6 Paper

The server had a scoreboard team with an invalid colour code (none colours, like m for strikethrough) There is nothing here we can do bar I guess magically stripping those codes. Or the server administrator removing them. I am semi in the camp of "not much we should/could do" here, but I'll leave this open for more team opinions.

Why not just sanitize it? It makes it easier for less experienced server admins who want to use these plugins and may not know how to diagnose these issues. It would be pretty easy to just put in a catch right?

rustic folioBOT
rustic folioBOT
#
codeHusky
[PaperMC/Paper] Issue opened: #12864 CraftAbstractInventoryView returns wrong Slot number in Crafter views

Like the Crafting Table, the Crafter menu uses different ID ordering than normal over the network. Unlike the Crafting table, the Crafter is weird.
https://minecraft.wiki/w/Java_Edition_protocol/Inventory#Crafter

Exact test conditions are checking getClickedInventory() instanceOf PlayerInventory, e.getClick().isRightClick(), as well as getCursor().getType() == Material.AIR, though I don't believe those are important.

The following code will fail with the current implementation, placing the item in the wrong place during an InventoryClickEvent.

InventoryClickEvent event; // example
// ... 
event.getInventory().setItem(event.getSlot(), new ItemStack(Material.DIAMOND));
// OR
event.setCurrentItem(new ItemStack(Material.DIAMOND));

<img width="377" height="350" alt="Image" src="https://github.com/user-attachments/assets/124ae44d-8384-4dcb-9755-8792c3b3b9a2" />

<img width="384" height="350" alt="Image" src="https://github.com/user-attachments/assets/4e3dab93-6389-4a87-a9cd-fa1d4a9692...

rustic folioBOT
#
BenchmarkMode
[PaperMC/Paper] New comment on issue: #12858 server crash

I am unsure how much we can do there.

Introducing a sync chunk load is what it is. It is already scheduled at blocking priority in the chunk system. The only potential "solution" here would be to consider clipping into an unloaded chunk a miss. That however obviously is diverging from normal vanilla behaviour and might have unexpected outcomes.

Players can't move into unloaded chunks, why should they be able to interact with them? I don't think any possible vanilla feature could break from this, even then, maybe make it configurable like the player unloaded chunk movement blocker

rustic folioBOT
#
codeHusky
[PaperMC/Paper] New comment on issue: #12864 CraftAbstractInventoryView returns wrong Slot number in Crafter views

I've implemented a fix for this on my own Folia fork that may be useful in resolving it properly
https://github.com/CallMeCarsub/Folia/blob/b22bc84294a3dcdbbbceefca58993ceeca732ba0/folia-server/paper-patches/features/0010-Fix-crafter-slots.patch

// The Crafter's result slot is indexed at the end rather than in first 10 slots
if(this.getType() == InventoryType.CRAFTER){
    if(slot == 45){
        // return the relevant slot (? untested)
        return 9;
    }
    slot += 5;
}
rustic folioBOT
#
electronicboy
[PaperMC/Paper] New comment on issue: #12854 A way to upgrade old NBT data by plugins

The problem is that we don't have a mechanism to say "this plugin requires X version of MC and we should blow up initialization if it doesn't", that means that your players are going to update their servers and anything that loads in between them starting on the new version and loading up the updated version of your plugin is going to basically be lost, so you'd still be maintaining logic to have to deal with the old data; The modding ecosystem is generally much more tightly defined against MC versions so has a real advantage there in terms of making sure that people upgrade properly

An API for moving custom_data stuff into the PDC does sound fine, it would just require somewhat fairly early registration support in oder to allow people to deal with data that might load before plugins have a chance to register

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
metabrixkt
[PaperMC/Paper] Issue opened: #12865 `Player#hasResourcePack()` is `false` if packs are loaded with `AsyncPlayerConnectionConfigureEvent`

Expected behavior

  1. A resource pack is loaded on AsyncPlayerConnectionConfigureEvent
  2. When a CraftPlayer instance is created, the resource pack status for that connection is written to CraftPlayer#resourcePackStatus
  3. Player#getResourcePackStatus() returns the proper status and Player#hasResourcePack() returns true if the pack loading succeeded

Observed/Actual behavior

  1. A resource pack is loaded on AsyncPlayerConnectionConfigureEvent
  2. When a CraftPlayer instance is created, the field CraftPlayer#resourcePackStatus stays null
  3. Player#getResourcePackStatus() returns null and Player#hasResourcePack() returns false until a pack is loaded during the game phase

Steps/models to reproduce

  1. Load a pack in an AsyncPlayerConnectionConfigureEvent event handler
  2. Call Player#getResourcePackStatus() and Player#hasResourcePack() when the player joins the game
  3. Get the values null and false respectively despite the pack being active on the client and the pack callback p...
rustic folioBOT
rustic folioBOT
#
notTamion
[PaperMC/Paper] New comment on pull request: #12826 Added `Entity#getInvulnerable` method

but also I don't want to over-explain the deprecation

feel free to give a bit more context in the deprecation message. I don't have the source infront of me right now but that method seemed pretty useless anyways (given it only checks for api invuln and one more thing afaik).

Btw i think you will need to add an override for craftlivingentity which uses the isInvulnerableTo method instead of isInvulnerableToBase so all the conditions of the entity apply. (don't have the source infront of me rn like i said so im not 100 % sure)

#
NonSwag
[PaperMC/Paper] New comment on pull request: #12826 Added `Entity#getInvulnerable` method

given it only checks for api invuln and one more thing afaik

It also checks whether the entity is dead, in creative mode, and other stuff

<img width="1025" height="157" alt="image" src="https://github.com/user-attachments/assets/e04dbfb0-6641-4803-a489-42d9fc73a0dc" />

Btw i think you will need to add an override for craftlivingentity which uses the isInvulnerableTo method instead of isInvulnerableToBase so all the conditions of the entity apply.

You are right, totally forgot about that.

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Emilxyz
[PaperMC/Paper] Pull request opened: #12867 feat(plugin): make Plugin extend Namespaced

This PR makes Plugin extend adventures Namespaced and implements the Namespaced#namespace() method in PluginBase.

Motivation

This allows passing a plugin instance as namespace into Key#key(Namespaced, String) (or anything that accepts a Namespaced instance). I laid out my personal use case for this in [discord](#paper-dev message) (although very very simplified), but I believe this could be useful to have in general.

Even if the benefit for my personal example is just a minor QOL thing, it seems as NamespacedKey is only actively being used in spigot code from before the hard fork and is supposed to be phased-out eventually (#10568 - although I'm unsure how up-to-date this is). So this change would also allow for an easier transition from NamespacedKey -> Key for developers.

Open questions

Should I also change the following methods in NamespacedKey to accept a Namespaced instead of a Plugin:
new NamespacedKey(Plugin, String) -> new NamespacedKey(Namespaced, String)
NamespacedKey#fromS...

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
NonSwag
[PaperMC/Paper] New comment on pull request: #11859 checkstyle configuration for API

Some things I spotted that might need fixing:

  • primitives (fields params and returns) require nullability annotations
  • void returns require nullability annotatins
  • @NonNull (nullability information in general) infront of the modifiers is allowed if it is on the same line as the method but should only be allowed AFTER the modifiers - example: @NonNull public static X is allowed but should be public static @NonNull X
  • deprecated (not marked for removal) class are still checked which seems redundant
  • there is no rule about the order of nullability annotation and final in method parameters, should it be @NonNull final or final @NonNull
  • package-private and interface methods should be allowed (or even required) to have the nullability annotation not on the same line as the target
rustic folioBOT
rustic folioBOT
#
SplatDaddyMC
[PaperMC/Paper] Issue opened: #12868 1.21.8-6: Ender pearls don't teleport you after a relog

Expected behavior

Ender pearls teleport the player after a relog.

Observed/Actual behavior

Ender pearls do not teleport the player after a relog.

Steps/models to reproduce

  1. Have legacy-ender-pearl-behavior set to 'true'
  2. Throw ender pearl in a stasis chamber
  3. Log out of the server
  4. Log back in
  5. Flip the pearl
  6. Observe no teleport

Plugin and Datapack List

No plugins or datapacks

Paper version

This server is running Paper version 1.21.8-6-main@782ce95 (2025-07-18T20:26:58Z) (Implementing API version 1.21.8-R0.1-SNAPSHOT)
You are running the latest version
Previous version: 1.21.7-10-6ea679e (MC: 1.21.7)

Other

Note: this does NOT occur if legacy-ender-pearl-behavior is set to 'false'. It only occurs when legacy-ender-pearl-behavior is set to 'true'. If the pearl is thrown while it's set to 'true', then you relaunch the server with it set to 'false', then flip the pearl, it still does not teleport you. This suggests that pearl data is permanen...

#
Machine-Maker
[PaperMC/Paper] New branch created: feature/update-configurate
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
BenchmarkMode
[PaperMC/Paper] Issue opened: #12871 block send logic seemingly ran twice for applyPhysics=false?

Expected behavior

Cratblock.setBlockState()

 if (applyPhysics) {
            return world.setBlock(pos, newState, net.minecraft.world.level.block.Block.UPDATE_ALL);
        } else {
            boolean success = world.setBlock(pos, newState,
                net.minecraft.world.level.block.Block.UPDATE_CLIENTS |
                    net.minecraft.world.level.block.Block.UPDATE_KNOWN_SHAPE |
                    net.minecraft.world.level.block.Block.UPDATE_SKIP_ON_PLACE);
            if (success && world instanceof net.minecraft.world.level.Level) {
                world.getMinecraftWorld().sendBlockUpdated(
                    pos,
                    oldState,
                    newState,
                    net.minecraft.world.level.block.Block.UPDATE_ALL
                );
            }
            return success;
        }
    }

 if ((flags & 2) != 0 && (!this.isClientSide || (flags & 4) == 0) && (this.isClientSide || chunkAt == null || (chunkAt.getFullStatus() != null```...
rustic folioBOT
#
BenchmarkMode
[PaperMC/Paper] New comment on issue: #12871 block send logic seemingly ran twice for applyPhysics=false?

the flags passed into sendBlockUpdated are completely unused, I can say pretty confidently that sendBlockUpdated is called twice with the exact same parameters (excluding flags)

// oldstate comes from this.getChunkAt(pos).setBlockState(pos, state, flags) (setblockstate returns replaced block state)

BlockState state = newState;
BlockState blockState = oldState;

 if ((flags & 2) != 0 && (!this.isClientSide || (flags & 4) == 0) && (this.isClientSide || chunkAt == null || (chunkAt.getFullStatus() != null && chunkAt.getFullStatus().isOrAfter(FullChunkStatus.FULL)))) { // allow chunk to be null here as chunk.isReady() is false when we send our notification during block placement // Paper - rewrite chunk system - change from ticking to full
                this.sendBlockUpdated(pos, blockState, state, flags); // pos, oldState, newState
            }


// oldstate comes from this.getNMS(), which calls this.world.getBlockState(this.position)
// new state identical to that which```...
#
ColsonPlayz
[PaperMC/Paper] Issue opened: #12872 Teleport flags not retaining passengers on cross-world travel

Expected behavior

I use teleport flag to retain passengers and passengers are kept when you teleport from 1 world to another. (it works fine for teleporting in the same world)

Observed/Actual behavior

It literally does nothing. Teleport effects called with the retain passengers teleport flag, if the player has passengers and they are being teleported to a new world, do absolutely nothing. I get no errors, no chat messages, the player just doesn't get teleported.

Steps/models to reproduce

  1. Create a new papermc server (I tested it on 1.21.4 and 1.21.8)
  2. Make a new paper plugin and somewhere in there (probably a command/event) teleport the player to a location in the nether (or any alternate world) using teleport flag retain passengers
  3. Watch as nothing happens

Plugin and Datapack List

<img width="962" height="95" alt="Image" src="https://github.com/user-attachments/assets/135dfe68-6385-45e7-bbf0-bde168a60a4e" />

Paper version

1.21.8-R0.1-SNAPSHOT

O...

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
tommy10606
[PaperMC/Paper] Issue opened: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

Expected behavior

I believe the dataconverter should be able to convert the data from json to nbt.

Observed/Actual behavior

Errors like the following just get spammed in console.

[12:37:11] [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt: Aquatic Annihilation
com.google.gson.JsonSyntaxException: com.google.gson.stream.MalformedJsonException: Use JsonReader.setStrictness(Strictness.LENIENT) to accept malformed JSON at line 1 column 10 path $
See https://github.com/google/gson/blob/main/Troubleshooting.md#malformed-json
at com.google.gson.JsonParser.parseReader(JsonParser.java:116) ~[gson-2.11.0.jar:?]
at com.google.gson.JsonParser.parseString(JsonParser.java:92) ~[gson-2.11.0.jar:?]
at ca.spottedleaf.dataconverter.minecraft.versions.V4290$1.convert(V4290.java:235) ~[paper-1.21.8.jar:1.21.8-6-782ce95]
at ca.spottedleaf.dataconverter.minecraft.datatypes.DynamicDataType.convert(Dynamic...

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
lynxplay
[PaperMC/Paper] branch deleted: feature/update-configurate
#
Malfrador
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

Essentially, you have an invalid item in the playerdata. The game expects JSON there, but it got a plain string ("Aquatic Annihilation"). Based on the string would guess its the name of an item. Could also be lore, content of a written book or similar though - the error is very unhelpful here.

Vanilla will fail in the same place. We can probably fix this, but it would be important to know the item got created like that in the first place so we can prevent it from happening again in the future.

Do you have any suspicion as to which plugin was originally used to create the "Aquatic Annihilation" item? If it is an item at all.
May have happened a few versions ago or similar, but any clue could be helpful.
Its not going to be the plugin in the stacktrace, thats just whatever tried to load the playerdata. The issue lies in how the item was created in the first place.

I've tried vanilla /give in various versions ranging from 1.13 to latest, and it will correctly refuse to create such...

rustic folioBOT
rustic folioBOT
#
lynxplay
[PaperMC/Paper] New comment on pull request: #12861 Add "redstone_related" material tag

Thank you for the PR and welcome to paper :partying_face: !

I am closing the PR as I don't think we want to pull this.
The creative mode category API might not be very complete but it should allow for some basic filtering/checking of materials against their category.
A new tag here just seems like a bandaid fix. We should expand the creative mode API instead (tho there are PRs for that).

Thank you again for the contribution!

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
TZ122
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

I am receiving the same error https://mclo.gs/E128mNZ - The item below was spawned in on a 1.21.1 (or 1.20 I can't exactly remember) server using the command below.

minecraft:give @a firework_rocket{Fireworks:{Flight:1,Explosions:[{Flicker:1b,Trail:1b,Colors:[I;11546150,16351261,15961002],FadeColors:[I;3847130,3949738,8991416]}]},display:{Name:'Party Popper Firework',"italic":false,Lore:['["",{"text":"Crate","italic":false,"color":"red"}]']},HideFlags:32}

rustic folioBOT
#
Malfrador
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

Okay I can reproduce it.

  1. Run /give @s minecraft:firework_rocket{display:{Name:'Party Popper Firework',"italic":false,Lore:['["",{"text":"Crate","italic":false,"color":"red"}]']},HideFlags:32,Fireworks:{Flight:1,Explosions:[{Flicker:1b,Trail:1b,Colors:[I;11546150,16351261,15961002],FadeColors:[I;3847130,3949738,8991416]}]}} on 1.20.4
  2. You will get a Firework Rocket named "Firework Rocket" visually, not "Party Popper Firework". The client correctly ignores the invalid name.
  3. However, the server does not. Running /paper dumpitem shows that Name did indeed get set to a plain string
  4. Update the server to Paper 1.21.8
  5. You get the MalformedJsonException.

Notably, vanilla DFU will just silently remove the Name property from the item instead of blowing up with an exception.

rustic folioBOT
rustic folioBOT
#
alexk201
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

I understand that the focus of the original PR was secret management. As mentioned in my PR description, the secret alone does not suffice to configure velocity though. And I am really wondering how other people managed to get paper running on k8s without the newly introduced environment variables. K8s ConfigMaps don't work as they are mounted read only. The first thing spigot does at startup is reading and then re-writing the yaml config to disk so it crashes. Two options remain: Env variables or a persistent volume that behaves as a regular disk. The first option is contained in this PR. The second option would be ok-ish too but it's sooo hacky: If you have servers that get discarded anyways (like minigame servers), you don't want to use a persistent volume. If you use a temporary filesystem instead, you first have to create the required configuration files which isn't possible using the initSettings command as mentioned before. IMHO, env variables are the best option for configuring...

#
alexk201
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

Coming from the Java Enterprise world (Application Servers, Quarkus, and Spring Boot), I might be a bit spoiled - we've had the Eclipse MicroProfile for several years now. One of its standout features is the MicroProfile Config API, which provides a flexible abstraction layer over various configuration sources. Most implementations support properties files, YAML, environment variables, and even more enterprise-specific options like JDBC, JNDI, and HashiCorp Vault - though many of those might not be particularly relevant for Paper.

I'm not sure how many people (besides me) are running Paper in Kubernetes environments, but it would be amazing to make everything configurable from any source. I'd be happy to contribute some time and share my experience from the Java Enterprise ecosystem to explore whether Eclipse MicroProfile (or a similar abstraction) could make sense for Paper.

That said, I'd love to hear the team's thoughts on this first. From what you mentioned, it seems like con...

rustic folioBOT
#
electronicboy
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

env-based config stuff has been on the "maybe cool to have", but it would need to be integrated in some manner into the configurate configuration library. However, such a concept just seems fairly limited, especially when it comes to non-primitive entries and such. We generally won't be migrating away from configurate unless something solves something it doesn't in a substantial manner, there just isn't the demand or resources for us to cater to these sorta setups, anybody at the scale of needing such envs is generally forking to cater for their needs anyways

#
Machine-Maker
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

I've thought about setting up something where simple primitives could be overriden by environment variables. I don't want to get crazy with it, where we have some crazy way env vars can set lists or full objects. But detecting simple enough primitive values, and generating an env var key based on its path should be doable.

Have to do it in such a way to merge it into the config node without writing it back out I think, that might be harder.

#
lynxplay
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

An alternative we could potentially investigate a bit further is supporting ro configuration files.
In general, our migrations and defaults don't need to be written back to file, just obviously makes editing
the files worse if keys are missing and the file is not on the latest version but that can be up to the administrators then.

I believe we currently don't fail if the paper configs cannot be written to?
We presumably error very loudly but it should startup and run the migrations on the in-memory representation.
Idk if that is applicable to spigot.yml and bukkit.yml, those use a different system rn.

#
electronicboy
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

RO config files are one of those weird ones where I think we shifted some concessions towards (but, this is one of the areas why I kinda supported it, even if it makes no sense for most setups and is often an early warning sign something is wrong)

afaik, k8s would let you expose a config file in the container from the ConfigMap or whatever it was, but I don't have the resources to waste time learning all of that stuff yet

#
alexk201
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

Unfortunately, paper does not ignore read only (in a k8s environment)

[19:11:54 ERROR]: Encountered an unexpected exception
org.spongepowered.configurate.ConfigurateException: []: java.nio.file.FileSystemException: /opt/paper/config/.7199899733415571904970619paper-global.yml.tmp -> /opt/paper/config/paper-global.yml: Device or resource busy
        at org.spongepowered.configurate.loader.AbstractConfigurationLoader.save(AbstractConfigurationLoader.java:211) ~[configurate-core-4.2.0-20250225.064233-204.jar:?]
        at io.papermc.paper.configuration.Configurations.trySaveFileNode(Configurations.java:113) ~[paper-1.21.7.jar:1.21.7-DEV-68488bd]

The server crashes and shuts down. Maybe it's overkill to migrate the entire file configuration to a different system. Everything that can be configured at runtime like game difficulty doesn't need to be exposed using some other configuration system. Properties that need to be set before server startup would be nice to be exposed some...

#
alexk201
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

In case anyone wants to reproduce it in k8s, here's a shortened version of my setup:

apiVersion: v1
kind: ConfigMap
metadata:
  name: paper-global-config
data:
  paper-global.yml: |
    _version: 29
    anticheat:
      [...]
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: paper
  labels:
    app.kubernetes.io/name: paper
spec:
  replicas: 1
  selector:
    matchLabels:
      app.kubernetes.io/name: paper
  template:
    metadata:
      labels:
        app.kubernetes.io/name: paper
    spec:
      containers:
        - name: paper
          image: my-registry/paper:1.21.7-26
          volumeMounts:
            - name: config-volume
              mountPath: /opt/paper/config/paper-global.yml
              subPath: paper-global.yml
      volumes:
        - name: config-volume
          configMap:
            name: paper-global-config
rustic folioBOT
#
okx-code
[PaperMC/Paper] Pull request opened: #12876 Optimise mob despawning

This PR introduces a spatial data structure used to find the nearest player for the purposes of despawning mobs.

Currently, every mob loops over every player per tick to determine if they are:

  1. Eligible for despawning mobs (not in spectator mode)
  2. The closest player to that mob

This is effectively an O(n ^ 2) operation with per-player mob spawning.

In this PR, we construct a K-d tree (k=3) with every despawn-eligible player every tick and use that to find the closest player to determine whether the mob should despawn. This is a very efficient data structure for nearest-neighbour searches (the implemented data structure is an adapted version from the book "Algorithms In A Nutshell").

I have also changed the despawnRanges config option to be represented as an EnumMap rather than a HashMap to make it more efficient.

I considered using the NearbyPlayers class, however it raises a couple complications that made me decide against using it in the end. It stores players by...

rustic folioBOT
#
Owen1212055
[PaperMC/Paper] New branch created: feat/disable-ear
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
notTamion
[PaperMC/Paper] New comment on pull request: #12780 Fixes rapid fire of 'EntityCrossbowLoadEvent' until player stops holding right click.

It's not cancellable

How is that? I am able to keep the event cancelled just fine.

just because two events are broken

PlayerReadyArrowEvent behaving like this is intentional see #12111.

Events being fired a lot does not mean they are broken. I do agree that for EntityCrossbowLoadEvent the expected behavior of cancellation is debatable though given plugins might rely on this to e.g. delay crossbow loading this just seems like a pointless api break. if you want to have the player stop using the crossbow you can call clearActiveItem in the listener.

rustic folioBOT
#
kennytv
[PaperMC/Paper] branch deleted: offhand-desync
rustic folioBOT
#
RandomTomato
[PaperMC/Paper] Pull request opened: #12878 Add `PlayerPreTeleportEvent`

Adds PlayerPreTeleportEvent to address various issues with teleporting mentioned in #12093 & #10168.

Currently there is no way to handle teleporting players cross-worlds with vehicles and passengers as the PlayerTeleportEvent fires after the checks for vehicles and passengers.

Implementing this event would allow for intercepting of the teleport before it occurs and allow for the correct handling of the vehicle/passengers.

#
RandomTomato
[PaperMC/Paper] Pull request opened: #12879 Add PlayerPreTeleportEvent

Adds PlayerPreTeleportEvent to address various issues with teleporting mentioned in #12093 & #10168.

Currently there is no way to handle teleporting players cross-worlds with vehicles and passengers as the PlayerTeleportEvent fires after the checks for vehicles and passengers.

Implementing this event would allow for intercepting of the teleport before it occurs and allow for the correct handling of the vehicle/passengers.

rustic folioBOT
#
electronicboy
[PaperMC/Paper] New comment on pull request: #12879 Add PlayerPreTeleportEvent

The goal really is to align with the vanilla teleport logic rather than the CB logic and having a bunch of hacks

Really iffy on these sorts of events. Adding in early mutation points for stuff they don't really need, like the Location, is kinda bleh. Nor should we really be returning mutable location objects in the API There is zero info on whether a teleport will succeed or not, which feels like it kinda misses the bar in terms of what it's trying to solve

rustic folioBOT
#
RandomTomato
[PaperMC/Paper] New comment on pull request: #12879 Add PlayerPreTeleportEvent

The goal really is to align with the vanilla teleport logic rather than the CB logic and having a bunch of hacks

Really iffy on these sorts of events. Adding in early mutation points for stuff they don't really need, like the Location, is kinda bleh. Nor should we really be returning mutable location objects in the API There is zero info on whether a teleport will succeed or not, which feels like it kinda misses the bar in terms of what it's trying to solve

hello ๐Ÿ‘‹๐Ÿป Thank you for your reply! I understand that the goal is to align with the vanilla logic. This was mentioned by Malfrador in #12093 a feature request that I made a while ago and we both spoke in.

I also agree with not having a bunch of hacks/work arounds, but if there is no intention of implementing/aligning with the vanilla teleport logic (as its been nearly two years since the last pull request #10173 with a similar goal was created). Is it possible to be implemented and then deprecated when moved over?...

#
Leguan16
[PaperMC/Paper] New comment on pull request: #12879 Add PlayerPreTeleportEvent

It's also questionable if such an event should be cancellable.

But yeah, when working on #10173 I more and more realised that such an event doesn't really make much sense in a way to justify adding it.

However, as stated and discussed in my PR a TeleportFailEvent might make more sense in this case as you could then prepare the entity for teleportation.

#
RandomTomato
[PaperMC/Paper] New comment on pull request: #12879 Add PlayerPreTeleportEvent

It's also questionable if such an event should be cancellable.

But yeah, when working on #10173 I more and more realised that such an event doesn't really make much sense in a way to justify adding it.

However, as stated and discussed in my PR a TeleportFailEvent might make more sense in this case as you could then prepare the entity for teleportation.

I am personally happy with either approach. ๐Ÿคท๐Ÿปโ€โ™‚๏ธ
Its just kind of a pain that currently there is no way to deal with this and hopefully some sort of a solution can be implemented.

rustic folioBOT
#
davidmayr
[PaperMC/Paper] Draft pull request opened: #12880 Allow to change despawnInPeaceful

This PR adds a method to check if an entity should be removed on the peaceful difficulty and adds a method to change the behavior. I considered a cancellable event, however it seems like that this check is done every tick, so I decided for this approach.

The value also gets persisted in the entity data. However, I'm not sure if the data should be persisted when set to Tristate#NOT_SET. E.g. VisualFire is persisted even when set to NOT_SET, but I'm not sure if saving that data for every mob is necessary.

On the NMS side I decided to add a new method (shouldActuallyDespawnInPeaceful) as running this method in the only two places it is used is much cleaner than modifying shouldDespawnInPeaceful in every entity that implements it (like Ghast, Monster, etc.)

My usecase for this is the following: We use Entities like Zombies to represent spawn locations etc. in our setup on our map building server. However, most maps are set to peaceful so we are limited in the amount of entities t...

rustic folioBOT
#
NoahvdAa
[PaperMC/Paper] New comment on issue: #12798 PlayerInventory#setItemInOffHand does not update the player inventory when viewing another inventory

This is a server issue also present in vanilla (reproduce with /item replace) and doesn't just apply to the offhand but to all equipment slots.
The reason it happens is because in the player tick method broadcastChanges is called on the player's containerMenu. When you have a container open, this is the container menu, otherwise it is the player's inventory.
The player inventory menu contains the slots for equipment, while the container menu does not.

Also calling broadcastChanges on the ServerPlayer's inventoryMenu seems to fix the issue but obviously also leads to double checking of the player inventory slots.

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
DreiFxn
[PaperMC/Paper] New comment on issue: #7021 Most rideable entity statistic trackers (xxxx_one_cm) do not track movement correctly

Adding to this as of 1.21.8 Paper #11

Happy Ghast distance stat is broken
(what I noticed with the horse, boat & happy ghast ... the stats keeps increasing even when I'm just sitting on it, but not moving at all the other rideables just count the distance wrong when moving)

Still broken are:

  • Strider (Strider distance is weirdly counting way to fast up ... but it actually stops counting when sitting on it but not moving)
  • Horse
  • Boat
  • Pig

Note: I know that this seems to be an long standing issue, just wanted to add what I figured out. Noticed since some custom achievments popped way to early.

  • Drei
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
zinck78
[PaperMC/Paper] New discussion: #12885 Item Display issues

Is your feature request related to a problem?

Currently there's an issue with displaying items in shopkeeper/villager trades. The output section like what you'll receive does not appear causing confusion for players. This occurs on both 1.21.7 & 1.21.8

Describe the solution you'd like.

I'm pretty uneducated in this stuff idk what we'd need to do to fix this issue. However I do believe it's something that needs to be taken care of. I'm not sure if it's a paper problem or a minecraft problem at this point.

Describe alternatives you've considered.

Using deluxemenus instead of shopkeepers.

Other

https://github.com/user-attachments/assets/e0e6df74-8b58-4107-b09f-1f492032e06f

Discussion Category: Ideas
#
tommy10606
[PaperMC/Paper] Issue opened: #12886 Change what classifies as latest version

Is your feature request related to a problem?

This is not related to a problem. This would simply make it easier for me to know if there are updates to paper within the same game version or updates to a new game version.

Describe the solution you'd like.

When I run /version, paper tells me I'm running the latest version. with some additional info. I think it would be useful if that was split into latest version of paper for version X vs latest version of paper. For example, "You are running the latest version of paper for 1.21.4 (232). You are not running the latest version of paper (1.21.8-11)."

I don't know exactly how easy this would be to implement or what the exact wording should be.

Describe alternatives you've considered.

Open https://papermc.io/downloads/paper and manually check.

Other

<img width="616" height="128" alt="Image" src="https://github.com/user-attachments/assets/b057e1a8-8d87-4793-bb61-a6f7a85b5d10" />

Notice it says I'm running the latest...

rustic folioBOT
#
lynxplay
[PaperMC/Paper] New branch created: bugfix/update-mache
rustic folioBOT
#
lynxplay
[PaperMC/Paper] branch deleted: bugfix/update-mache
rustic folioBOT
#
professorparabellum
[PaperMC/Paper] Issue opened: #12888 1.8.8 ViaRewind keepalive kick

Expected behavior

players on 1.8.8 are able to join and play normally, without getting kicked when their game stutters

Observed/Actual behavior

this recent paper patch is very ruthless with players on legacy clients as any sort of delay or stuttering (e.g. high ping, low fps) will cause them to get kick instantaneously after they send a keepalive packet.

Youtube demo: https://youtu.be/Vq8uhkEsET4

it also has no care for if its actually the player or the netty thread being congested, since its time between player and main thread not player and server.

Steps/models to reproduce

  1. Join a server with ViaVersion, ViaBackwards, ViaRewind on a 1.8.8 client
  2. Turn on 3D Anagylph to make your game stutter/freeze on purpose
  3. Wait 2 or 3 seconds
  4. Get kicked

Plugin and Datapack List

[03:57:09 INFO]: Paper Plugins (3):
[03:57:09 INFO]:  - HuskSync, ScPaper, towns
[03:57:09 INFO]: Bukkit Plugins (28):
[03:57:09 INFO]:  - *Achievem*```...
rustic folioBOT
#
jtJava
[PaperMC/Paper] Issue opened: #12889 Projectiles stop tracking when players move into them

Expected behavior

The projectile should keep tracking, instead of phasing to where it lands.

Observed/Actual behavior

https://youtu.be/pxklrZbF_R4

Steps/models to reproduce

Shoot arrows while moving forward on a 1.21.8 paper server with no plugins, the arrows will desync. This bug does not happen on 1.21.8 vanilla

Plugin and Datapack List

None

Paper version

This server is running Paper version 1.21.8-11-main@a5f2f61 (2025-07-22T08:53:33Z) (Implementing API version 1.21.8-R0.1-SNAPSHOT)
You are running the latest version
Previous version: 1.21.4-DEV-25e1915 (MC: 1.21.4)

Other

Maybe related to https://github.com/PaperMC/Paper/issues/12200

rustic folioBOT
rustic folioBOT
#
lightPlugins
[PaperMC/Paper] Issue opened: #12890 ClientboundSetCursorItemPacket / ServerboundContainerClickPacket triggering twice

Expected behavior

one time Trigger

Observed/Actual behavior

Triggering twice

Steps/models to reproduce

When I left-click an item in my inventory so that it's on the cursor, and then left-click on a free slot while holding the mouse button, slightly move the mouse, and then release the button, the ServerboundContainerClickPacket and ClientboundSetCursorItemPacket packet is triggered twice.

Plugin and Datapack List

  • eco (my fork, try to fix Ghost Items)
  • EcoItems

Paper version

paper 1.21.8 build #11

Other

No response

#
lynxplay
[PaperMC/Paper] New comment on issue: #12890 ClientboundSetCursorItemPacket / ServerboundContainerClickPacket triggering twice

What the client sends is up to the client?
The drag is simply implemented like this.
How is this a paper issue or even remotely something we could fix?

Checking against the vanilla, dragging before placing in the same slot emits a total of three click packets.

Can you elaborate here, maybe I am missing something, otherwise this issue can be closed as this works as intended.

rustic folioBOT
#
RomaYT
[PaperMC/Paper] Issue opened: #12891 The latest version of Paper does not display PlayerCount correctly

Expected behavior

The text "Hello" should be rainbow.

Observed/Actual behavior

I downloaded the plugin, installed it and changed PlayerCount

Steps/models to reproduce

  1. Downloaded the latest version of the kernel https://papermc.io/downloads.

  2. Threw it into the folder.

  3. Threw the plugin AdvancedServerList-Paper-5.7.0-b2 into the plugins folder

  4. Went to advancedserverlist >> profiles >> default.yml

Changed the file, adding the following to it.

profiles:
  - motd:
      - "<bold>Hello<reset> World!</reset>     "
      - ""
    playerCount:
      text: '<bold>Hello<reset> World!</reset>     '
      hover:
        - ""
  - motd:
      - "<rainbow><bold>Hello<reset> World!</reset>     "
      - ""
    playerCount:
      text: '<rainbow><bold>Hello<reset> World!</reset>     '
      hover:
        - ""

and when I launched it I got an incorrect PlayerCount display.

Plugin and Datapack List

[20:43:45 INFO]: โ„น Server Plugins (1):
[20:43:45 INFO]: Paper Plugins:...

#
RomaYT
[PaperMC/Paper] New comment on issue: #12891 The latest version of Paper does not display PlayerCount correctly

ะะต ะฒะธะถัƒ ะทะดะตััŒ ะฟั€ะพะฑะปะตะผั‹. ยซPlayerCountยป โ€” ัั‚ะพ ัั‚ั€ะพะบะพะฒะพะต ะฟั€ะตะดัั‚ะฐะฒะปะตะฝะธะต ะฒะตั€ัะธะธ ัะตั€ะฒะตั€ะฐ, ะบะพั‚ะพั€ะฐั, ะบะฐะบ ัะบะฐะทะฐะป ะฐะฒั‚ะพั€ ะฟะปะฐะณะธะฝะฐ, ะฝะต ะฟะพะดะดะตั€ะถะธะฒะฐะตั‚ ั‚ะตะบัั‚ะพะฒั‹ะต ะบะพะผะฟะพะฝะตะฝั‚ั‹. ะญั‚ะพ ะฝะต ะฟั€ะพะฑะปะตะผะฐ Adventure, ะฐ ะฟั€ะพัั‚ะพ ยซะพะณั€ะฐะฝะธั‡ะตะฝะธะตยป ะบะปะธะตะฝั‚ะฐ, ะตัะปะธ ะผะพะถะฝะพ ั‚ะฐะบ ะฒั‹ั€ะฐะทะธั‚ัŒัั.

But then why on proxy (velocity) It works better? Although the plugin is the same..

rustic folioBOT
rustic folioBOT
#
vadcx
[PaperMC/Paper] Pull request review comment: #12876 Optimise mob despawning

You need not to worry about this unless the size difference is going to be huge. It will not in this case. I wrote down an explanation below (out of curiosity, chatgpt spew out the exactly same given a correct lead question)

jvm source

  1. Empty ArrayList = backing array literally empty object
  2. First addition: grow() aka grow(size=0 + 1) -> new array of size DEFAULT_CAPACITIY = 10.
  3. Adding 11th element to size=10 array increases capacity by oldCapacity>>1 aka ~0.5x aka +5 here
  4. Expanding from 15 adds +7
  5. Expanding from 22 adds 11
  6. Expanding from 33 adds 16
  7. To fit our potential 50th player, expand from 49 to 73 by adding 24.
  8. Thus final array elements = 50, but backing array has 73 slots, because it doesn't know when you'll stop.

PS: I learned to love digging thanks to a certain Shipilev. I want to further spread this approach :)

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Goinging
[PaperMC/Paper] Issue opened: #12893 Add paper global option for hiding player health information

Is your feature request related to a problem?

By default players are sent health packets for other players around them as well as animals. Players can download mods which take those packets and use them to display players/animal health. This information gives players an unfair advantage in pvp and is just extra information vanilla players don't get.

Describe the solution you'd like.

A new paper global setting which blocks these packets seems like something that would be at home in paper. Some servers display health already with TAB so it isn't something that all servers need. However, for servers which don't display player or mob health like how it is in vanilla, these mods are an unfair advantage.

This config setting would be in the anticheat section of paper global and is very similar to the existing obfuscation settings. The config would look something like this.

enable-health-obfuscation: false
overrides:

also-obfuscate: []
dont-obfuscate: 
- wolf
- iron golem

A...

rustic folioBOT
#
Supermarcel10
[PaperMC/Paper] Issue opened: #12894 `StackOverflowError: null` thrown on player.closeInventory()

Stack trace

[07:48:28 ERROR]: Could not pass event InventoryCloseEvent to TestPlugin v1.0.0
java.lang.StackOverflowError: null
        at java.base/java.util.Objects.equals(Objects.java:64) ~[?:?]
        at java.base/jdk.internal.util.StrongReferenceKey.equals(StrongReferenceKey.java:69) ~[?:?]
        at java.base/java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:940) ~[?:?]
        at java.base/jdk.internal.util.ReferencedKeyMap.get(ReferencedKeyMap.java:209) ~[?:?]
        at java.base/jdk.internal.util.ReferencedKeySet.get(ReferencedKeySet.java:173) ~[?:?]
        at java.base/java.lang.invoke.MethodType.makeImpl(MethodType.java:401) ~[?:?]
        at java.base/java.lang.invoke.MethodType.methodType(MethodType.java:382) ~[?:?]
        at java.base/java.lang.invoke.MethodType.dropParameterTypes(MethodType.java:692) ~[?:?]
        at java.base/java.lang.invoke.LambdaFormEditor.bindArgumentType(LambdaFormEditor.java:562) ~[?:?]
        at java.base/java.lang.inv```...
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Lulu13022002
[PaperMC/Paper] New comment on issue: #12819 TrialSpawners lose SpawnData/SpawnPotentials if a BlockTransformer has been registered for the Structure placing it

Not really some change in this commit doesn't apply to paper, like the vault one, for the banner this seems just redundant to clone the list twice since Banner#getPatterns already clone the underlying list. The only relevant change is the sign loading but it's not the issue reported in this ticket nor the main cause (fixed in the mentioned PR now).

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Lauriichan
[PaperMC/Paper] New comment on issue: #12819 TrialSpawners lose SpawnData/SpawnPotentials if a BlockTransformer has been registered for the Structure placing it

Great, thank you!

<img width="658" height="121" alt="Image" src="https://github.com/user-attachments/assets/69ea750e-0410-44c7-9bcf-f538cb54065f" />
However I noticed this part right here stayed the same, are these configs immutable?
If so this should be fine however if not then they should not be referencing the configs of the state that is being copied

rustic folioBOT
rustic folioBOT
#
DerMistkaefer
[PaperMC/Paper] New comment on issue: #12844 1.21.7: PlayerResourcePackStatusEvent not fired

This was moved from ServerCommonPacketListenerImpl to ServerConfigurationPacketListenerImpl as part of
https://github.com/PaperMC/Paper/commit/7f609243907131c37d457aa367c04efbbd10f022

However, ServerConfigurationPacketListenerImpl no longer provides access to a player.
Maybe this should be done later in handleConfigurationFinished, or should we consider introducing new events specifically for the configuration phase?

The use case is to track which resource pack a player has applied on the server, so it can be properly unloaded when the player switches to a different server within the server-network.

rustic folioBOT
rustic folioBOT
#
lynxplay
[PaperMC/Paper] branch deleted: feat/disable-ear
rustic folioBOT
rustic folioBOT
#
NoahvdAa
[PaperMC/Paper] New comment on issue: #12893 Add paper global option for hiding player health information

A few years ago I planned on making this a Paper config option, but when implementing it I found out that it's a massive pain to implement in the server directly. The only 'clean' way is to hook into the same system a plugin would use, which isn't really designed to be used by server internals.
After some internal discussion we decided that this is something that should be done by a plugin because we didn't want to maintain a messy implementation when plugins could handle it way cleaner.
That said, that was a few years ago, so maybe this is something that we do want to directly include now. I'll leave the issue open so others can comment their thoughts.

ps: I'll look into updating HealthHider :)

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
aerulion
[PaperMC/Paper] Issue opened: #12899 PlayerTeleportEvent.TeleportCause inconsistencies regarding (dis-)mounting entities

Expected behavior

Being able to properly distinguish teleports caused by (dis-)mounting entities.

Observed/Actual behavior

When listening to the PlayerTeleportEvent and checking the teleport cause, the result is different to what I expected. When dismounting any ridden entity, the teleport cause is always TeleportCause#UNKNOWN instead of TeleportCause#DISMOUNT. Also, the Javadoc for the dismount cause states that it is 'caused by a player exiting a vehicle', but I would find it more logical if it covered dismounting all entities, since all entities can be ridden nowadays using the /ride command.
Another odd behavior is that some animals, such as horses, donkeys, mules and llamas, trigger a TeleportCause#UNKNOWN when mounting, whereas other entities, such as saddled pigs, boats, etc., do not trigger the TeleportEvent at all. It would be useful to have a TeleportCause#MOUNT for this use case.

Steps/models to reproduce

  1. Listen to the PlayerTeleportEvent
  2. Mount and dis...
rustic folioBOT
#
electronicboy
[PaperMC/Paper] New comment on issue: #12809 Water Creatures seem to despawn unexepectly

having a name is irrelevant, it's all down to "does this entity have the persistence required flag set or is this a mob which should be considered persistent by the game" - i.e. nametags explcitly set the persistence required flag when setting a name on the entity

Water mobs are not considered persistent by default and so require the flag to stick around; it is also worth noting that the despawn logic works by finding the closest player to the entity (who can affect mob spawning), if the cloest person to the entity is outside of the range of the mob, it will be despawned, otherwise, if it finds nobody, or somebody is within the soft despawn range, it does nothing

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
lynxplay
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

From a glance at the offending code, we only support proper denied writes, e.g. missing write permissions for the server user.
You could hence probably fix this by defining the files as read-only via
defaultMode: 0444 in your volumes[0].configMap.

With semi obvious node that mounting in a single file is a bit of a hack so you might also need to define the world defaults. This makes this technique kinda questionable for the spigot and bukkit config but yea.
That should throw you an agressive warning about not being able to write but it should correctly let you start the server.

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Privatech38
[PaperMC/Paper] Issue opened: #12901 Dropping whole item stacks (Ctrl+Q) doesn't work in crafting inventories

Expected behavior

Ctrl+Q (default mapping) should drop the whole item stack in a crafting result (for the sufficient provided materials).

Example of this being turning a stack of ink sacks to a stack of black dye. Ctrl+Q on the resulting black dye should turn all ink sacs to black dye and drop them.

Observed/Actual behavior

Ctrl+Q only drops and consumes materials for one item no matter the recipe.

Steps/models to reproduce

  1. Survival mode,
  2. Open either player's crafting inventory or a crafting table's crafting inventory
  3. Place enough materials for a recipe to make more than one item from that recipe,
  4. Press Ctrl+Q on resulting item

Plugin and Datapack List

plugins
[00:20:49 INFO]: โ„น Server Plugins (0):
datapack list
[00:20:57 INFO]: There are 3 data pack(s) enabled: [vanilla (built-in)], [file/bukkit (world)], [paper (built-in)]
[00:20:57 INFO]: There are no more data packs available

Paper version

plugins
[00:20:49 INFO]: โ„น Server Plugins (0)...

rustic folioBOT
#
MrPowerGamerBR
[PaperMC/Paper] Issue opened: #12902 Items given using Adventure with custom colors desync in the inventory when interacted with

Expected behavior

When clicking the item on the inventory, the item should not desync

Observed/Actual behavior

https://github.com/user-attachments/assets/3a832a42-7d13-46df-87af-64f625983441

Steps/models to reproduce

        player.inventory
            .addItem(
                ItemStack(Material.DIAMOND)
                    .apply {
                        setData(
                            DataComponentTypes.LORE,
                            ItemLore.lore().addLine {
                                Component.text("hewwo!!")
                                    .decoration(TextDecoration.ITALIC, false)
                                    .color(TextColor.color(24, 230, 24)) // this should be any color that has a letter in the color's hex code
                            }
                        )
                    }
            )

Plugin and Datapack List

[01:00:42 INFO]: โ„น Server Plugins (110):
[01:00:42 INFO]: Bukkit Plugins:
[01:00:42 INFO]:  -```...
rustic folioBOT
#
MrPowerGamerBR
[PaperMC/Paper] New comment on issue: #12902 Items given using Adventure with custom colors desync in the inventory when interacted with

Thanks to JavierFlores09 on Discord, the issue was tracked down! It is a bug on Adventure on the TextColor::asHexString method.

Here's the workaround while the bug is not fixed in Adventure.

    static final Codec<TextColor> TEXT_COLOR_CODEC = Codec.STRING.comapFlatMap(s -> {
        if (s.startsWith("#")) {
            @Nullable TextColor value = TextColor.fromHexString(s);
            return value != null ? DataResult.success(value) : DataResult.error(() -> "Cannot convert " + s + " to adventure TextColor");
        } else {
            final @Nullable NamedTextColor value = NamedTextColor.NAMES.value(s);
            return value != null ? DataResult.success(value) : DataResult.error(() -> "Cannot convert " + s + " to adventure NamedTextColor");
        }
    }, textColor -> {
        if (textColor instanceof NamedTextColor named) {
            return NamedTextColor.NAMES.keyOrThrow(named);
        } else {
            return textColor.asHexString().toUpperCase(); // This is the```...
rustic folioBOT
#
alexk201
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

Thanks for investigating on the topic of Kubernetes ConfigMaps even though this is not your main concern :)
Unfortunately, the defaultMode doesn't prevent the error. After adding it, the permissions are looking good:

$ kubectl exec <my-pod> -- ls -la config
total 4
drwxr-xr-x. 2 root root   30 Jul 29 09:27 .
drwxr-xr-x. 1 root root   32 Jul 29 09:27 ..
-r--r--r--. 1 root root 3700 Jul 29 09:25 paper-global.yml

This still results in the same stack trace. Problem is that the underlying exception is a java.nio.file.FileSystemException instead of a java.nio.file.AccessDeniedException.

I should've added the complete stack trace earlier...

[09:27:51 ERROR]: Encountered an unexpected exception
org.spongepowered.configurate.ConfigurateException: []: java.nio.file.FileSystemException: /opt/paper/config/.2390207891523211899288440paper-global.yml.tmp -> /opt/paper/config/paper-global.yml: Device or resource busy
	at org.spongepowered.configurate.loader.AbstractConfi```...
#
lynxplay
[PaperMC/Paper] New comment on pull request: #12834 configure velocity proxy using env variable

Interesting, I did validate my claim at least on a local kind cluster before replying o.O but yea, permissions look good. An alternative we could consider, that might be a lot easier for us to implement, is allowing definition of overrides to the config yaml files via secondary filesets.
I am thinking java -jar server.jar --paper.global.values=/some/path/not/in/server/folder which can then directly be pointed at a config map mount point.

Limiting the overriding to yaml files would solve the "how do I define a map of lists of maps in env variables" problem by simply ignoring it and should be easily implemented via the existing merge functionalities in configurate.
Those files could then 100% be read only with the server not even attempting a write-back for migration and simply ignoring outdated keys during the merge.

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
epilepticseizureee
[PaperMC/Paper] Issue opened: #12904 Doesn't work player.customName(...)

Expected behavior

My code:
`public void setAdminName(Player player, int adminLvl) {
String name = player.getName();
String displayName = switch (adminLvl) {
case 0 -> "&f" + name;
case 1, 2 -> "&2" + name + "&r \uE005";
case 3, 4, 5, 6 -> "&c" + name + "&r \uE006";
default -> "&e" + name + "&r \uE007";
};

    player.playerListName(Component.text(Common.colorize(displayName + " &7[" + Objects.requireNonNull(getPlayerData(player)).getDynamicId() + "]")));
    player.customName(Component.text(Common.colorize(displayName + " &7[" + Objects.requireNonNull(getPlayerData(player)).getDynamicId() + "]")));
    player.displayName(Component.text(Common.colorize(displayName)));
}`

    player.customName(Component.text(Common.colorize(displayName + " &7[" + Objects.requireNonNull(getPlayerData(player)).getDynamicId() + "]"))); - not change my name tag head

Observed/Actual behavior

player.customName...

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Kaldoron
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

We had the same Error on our Server and i was able to track down the conflicting Items. In our Case it have been different Items that had been set up with a Colorized Name and Lore. Originaly the Items had been provided with a Give Command over Boss Shop. The Users that have been triggering this, have not been online for Years, but i cant see a Pattern of like "Oh its the change to this Version". I used the Errordetails and NBT Studio to search through the Playerdata Files and removed the 4 Usersfiles that caused this. Maybe that Helps you on your further Investigations.

rustic folioBOT
rustic folioBOT
#
TZ122
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

We had the same Error on our Server and i was able to track down the conflicting Items. In our Case it have been different Items that had been set up with a Colorized Name and Lore. Originaly the Items had been provided with a Give Command over Boss Shop. The Users that have been triggering this, have not been online for Years, but i cant see a Pattern of like "Oh its the change to this Version". I used the Errordetails and NBT Studio to search through the Playerdata Files and removed the 4 Usersfiles that caused this. Maybe that Helps you on your further Investigations.

Thank you for the reply. I tried NBT editor/NBT studio. My world folder has 300,000 player files and the application crashes trying to load the save. It seems I am stuck until this is fixed.

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
EvertBt
[PaperMC/Paper] Issue opened: #12907 Expose MapPallette.colors, Add MapPallette.getNearestColor

Is your feature request related to a problem?

Yes, recently all MapPallette methods like matchColor have been deprecated, with the only info received was to 'use color-related methods'. I have been unable find any of these mentioned color related methods.

I require the functionality to be able to match a given color to the closest map color.

Describe the solution you'd like.

  • expose MapPallette.colors, perhaps through MapPallette.getColors(), so that all available Colors can be known by plugins.

  • add a method like Color MapPallette.getNearestColor(Color c)

Describe alternatives you've considered.

MapPallette.colors could be accessed with reflection, and from there on we could write our own algorithm to match to the nearest color. But this does not seem like good practice, especially because the server already has all of this logic internally.

Other

No response

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
kezz
[PaperMC/Paper] Pull request opened: #12908 Adventure 4.24.0

Ready to go now, PaperAdventure#FLATTENER todo will be configurable with the new SPI system for overriding defaults when 4.24.1 is released but don't wait up on that as this fixes #12902 and flattener bugs probably reported somewhere but I can't find them.

rustic folioBOT
rustic folioBOT
#
davidmayr
[PaperMC/Paper] Issue opened: #12909 LEAD_BREAK sound plays even when EntityUnleashEvent is canceled

Expected behavior

The sound does not play when the event is canceled

Observed/Actual behavior

The unleash sound effect gets played repeatedly https://www.youtube.com/watch?v=LnWmlY6LeDE

Steps/models to reproduce

  1. Cancel the EntityUnleashEvent
  2. Leash to an entity
  3. Distance yourself from the entity so the leash snaps
  4. observe the sound beeing played every tick

Plugin and Datapack List

No datapacks
Test plugin

Paper version

[23:42:15 INFO]: Checking version, please wait...
[23:42:15 INFO]: This server is running Paper version 1.21.8-20-main@ec9c83f (2025-07-30T17:54:07Z) (Implementing API version 1.21.8-R0.1-SNAPSHOT)
You are running the latest version

Other

In net.minecraft.world.entity.Leashable the following is beeing run on line 174:

                if (d > entity.leashSnapDistanceOrConfig()) { // Paper - Configurable max leash distance
                    level.playSound(null, leashHolder.getX(), leashHolder.getY(), leashHolder.getZ(), Sou```...
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Hugo5000
[PaperMC/Paper] Issue opened: #12911 VehicleExitEvent is called after teleporting when done via performCommand while the player is sneaking

Expected behavior

VehicleExitEvent being called before being teleported via player.performCommand("minecraft:tp ~5 ~ ~") in the PlayerInputEvent for when the player is sneaking

Observed/Actual behavior

when doing player.performCommand("minecraft:tp ~5 ~ ~") inside the PlayerInputEvent with event.getInput().isSneak() the player already teleported when checking the location via the VehicleExitEvent and therefore area limitations aren't working.

Steps/models to reproduce

    @EventHandler(ignoreCancelled = true, priority = EventPriority.MONITOR)
    public void onPlayerInBoatGetsDamaged(PlayerInputEvent e) {
        if(e.getInput().isSneak()) e.getPlayer().performCommand("tp ~5 ~ ~");
    }

    @EventHandler(ignoreCancelled = true, priority = EventPriority.LOWEST)
    public void onPlayerTryExitBoat(VehicleExitEvent e) {
        // this doesn't cancel when teleporting via the performCommand above since the location already changed
        if(e.getVehicle().getLocati```...
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Suisuroru
[PaperMC/Paper] Pull request opened: #12912 fix: fix endermite spawn position

record by: An ender pearl has a 5% chance to spawn an endermite when it lands. This is the only way through which endermites can spawn, without using cheats. The endermite spawns at the player's position when the pearl landsโ€Œ[Java Edition only], or at the pearl's landing siteโ€Œ[Bedrock Edition only].

in the past the endermites will spawn at the position player teleported instead of old position becuse craftbukkit move teleport behind to create event, so it spawn in wrong place.

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Krakenied
[PaperMC/Paper] New comment on issue: #10430 BlockPistonExtendEvent doesn't fire when nothing is pushed

Rebased it directly:
0003-Paper-PR-9258.patch

However (as mentioned previously in the PR and also as stated in the docs) getDirection returns the direction the blocks (and the piston head) are moved in. With the rebased patch, the direction for retract without blocks is always the piston direction itself which is not correct behaviour. The current API returns the piston-opposite direction for sticky piston retracting with blocks.

The expected solution would be to return the direction the blocks are moved in, as the current API is used by many plugins, like for example WorldGuard: https://github.com/EngineHub/WorldGuard/blob/f8415edfcd505c0d7f998261568bc082f8db0877/worldguard-bukkit/src/main/java/com/sk89q/worldguard/bukkit/listener/EventAbstractionListener.java#L409-L419

The patch adapted to the expected behaviour:
0003-Paper-PR-9258.patch

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
EvertBt
[PaperMC/Paper] Issue opened: #12916 Allow access to the pixel colors of a map

Is your feature request related to a problem?

Yes

Describe the solution you'd like.

A way to access all colors/pixels of a map, so that it can be retrieved for database storage or cross server synchronization.

Describe alternatives you've considered.

The current way i do this is by using reflection to retrieve the buffer field from the maps MapItemSavedData#vanillaRender.

This field returns a byte[], which represents all colors stored as indexes, but since usage of the byte indexes is recently deprecated in the API, all access to byte -> color related methods are now scheduled for removal. So i will be unable to convert the retrieved byte[] into proper colors to for example render a map from one server on another, or save the pixel data.

After the removal, i am unsure how to continue development without access to map pixel data

Other

No response

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Kaldoron
[PaperMC/Paper] New comment on issue: #12873 [Server thread/ERROR]: [ca.spottedleaf.dataconverter.minecraft.versions.V4290] Failed to convert json to nbt:

I had to separate my Userfiles in Packages too but i can tell you, it's not a recent User that is bringing the Problem. I could track down the corrupted Files to the Years 2023 and 2018. I just packed all my Playerdata Files into Years according to the Lastchanged Status. It will be quite the Task on your Side with 300k+ Files, but for me it all was a Matter of "How badly do you want to Update befor your Holidays start" ;)

rustic folioBOT
#
Astralchroma
[PaperMC/Paper] Issue opened: #12919 Remove @Internal from Note.getId()

Is your feature request related to a problem?

The ID value represents a block state property and is used as part of the type's constructor which is not marked as @Internal. It is not some unreliable internal magic value, there are reasonable justifications as to why someone may need access this value.

Describe the solution you'd like.

Remove the @Internal annotation from Note.getId()

Describe alternatives you've considered.

Using it anyway, which I can and will do. However as the value isn't really internal, there is no reason it should be marked as such.

Other

No response

rustic folioBOT
rustic folioBOT
#
Astralchroma
[PaperMC/Paper] New comment on issue: #12919 Remove @Internal from Note.getId()

If the id is considered to be definitely unstable, then why is there a public constructor not marked as @Internal? That seems like an API inconsistency.

Additionally, from a quick glance at the wiki https://minecraft.wiki/w/Note_Block#Data_history, the representation of Note likely has not changed since 1.13, and before then it hadn't changed since it's introduction. It's safe to say it's extremely unlikely that Mojang would change it in a way that would break the meaning of the id.

#
electronicboy
[PaperMC/Paper] New comment on issue: #12919 Remove @Internal from Note.getId()

Those aren't really "user facing", the debug screen shows low level representations of stuff, not always values which are actually exposed within the game UI; the give/fill command has consistently changed with stuff over the years, we have no idea what mojang is likely to do when it comes to avenues like data driving around this logic, etc. The API has setters for Note rather than expecting you to pass in the raw ID for a reason.

rustic folioBOT
#
charles19132
[PaperMC/Paper] Pull request opened: #12920 Make telemetry opt-in

Many projects have gotten in heat for having opt-out telemetry. For example, The Fedora Project had a large amount of criticism due to opt-out telemetry.

bStats appears to be largely a third-party tracker. Currently, Paper users do not give any explicit consent to enable bStats, which is a third-party tracker. Paper also sends a server-specific identifier. Unless this telemetry is legitimate interest, this is largely illegal in the European Union.

Many Minecraft servers run on a personal PC and then shared using port redirection software or on a LAN. This makes the issue much worse than simply a containerized server sending data from a meaningless datacenter IP address.

It is in Paper's best interest to make this telemetry opt-in. If a plugin developer really wants the data, they can likely detect that bStats is off and then prevent the plugin or server from being fully loaded, or implement their own telemetry.

rustic folioBOT
#
Malfrador
[PaperMC/Paper] Pull request opened: #12921 Add bStats notice to onboarding message

Paper collects metrics via bStats. As such, we should generally follow GDPR.

The data collected falls under the legimate interest clause (pseudonymous, technical data, reasonable expectation, not overriding the fundamental rights and freedoms of the data subject), so consent is not required.

However, it still is a requirement to inform the users of the data collection and of how they can opt-out (Transparency).

The onboarding message is a good place for this, without spamming the regular startup log with information 99% of users do not care about. Showing the message once should be sufficient to inform the user.

I've added the following message below the onboarding message:

This server is collecting anonymous statistics for PaperMC via bStats.
This helps us improve our software. To disable, set 'enabled: false' in /plugins/bStats/config.yml
For more details on what data is collected, see https://bstats.org/privacy-policy

Alternatively we could add a note with...

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
#
IlIlIlIIlIlIlIl
[PaperMC/Paper] New comment on issue: #12529 Items outside activation range in water streams are lagging behind

I can confirm that this is an issue that also affects 1.21.5. Items flowing within a water stream quickly lose positional accuracy outside of a short distance from the player. This significantly interferes with basic item moving mechanics and disrupts even simple machines. An option to disable/alter this EAR behavior to restore item movement consistency would be very appreciated.

In spigot.yml you can set

entity-activation-range:
  misc: 16

to -1 to disable it, restoring the vanilla behaviour

rustic folioBOT
rustic folioBOT
rustic folioBOT
#
Warriorrrr
[PaperMC/Paper] New comment on issue: #11440 Collidable

setCollidable and collidable exemptions are purely server concepts, the player doesn't know it's not supposed to collide with a certain entity. The only way I can see that this could be fixed would be for the server to send team packets with collision set to never, but when dealing with plugins that may do the same it could become quite an ordeal.

rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT
rustic folioBOT