When will /recipe be back?
boredhero opened this issue · 28 comments
Hi, on b499 /recipe is temporarily disabled. Was wondering why and when this will be fixed?
@md678685 Any possible update on bringing /recipe back with an option to disable 1.12's recipe book? My players still far prefer being able to look up recipes by a command with the name of the item they want to craft. :(
@mibby As far as I know, Spigot haven't updated the API to allow us to fix the bug nor have they implemented a way to disable the recipe book.
/recipe is disabled on 1.12 only because of a dupe bug. The 1.12 recipe book should be sufficient for finding recipes anyway.
I'd argue that the recipe book isn't that sufficient. 1.) You have to go into survival mode to use it. 2.) You need to place and open a crafting table just to use it for items that use more than 4 slots to craft. Would it be possible to add a config option to put /recipe back and just leave a warning about it being bugged in 1.12?
I've disabled the recipe book on my server due to (yet another) dupe bug with it my players discovered. A toggle to enable/disable "/recipe" would be amazing.
I've got the same question. With mojang fixing the bug in 1.12.1, it seems EssentialsX is still disabled in build 514.
The deobfuscated NMS version didn't change as far as I can tell, so the detection behaviour won't change.
Is there any other way to detect it? I'd really really like to have /recipe back.
If the dupe bug can't be replicated in 1.12.1, then it's probably safe to just remove the check and tell people to update to 1.12.1, but I haven't yet been able to test it.
I have a lot of custom recipes, for which I have aliases. For example, "/recipe miner" would be an alias of "/recipe dispenser 4". I can't do that with the default 1.12 recipe book. =(
Bump. I'd also like to see recipe return, considering the dupe issue as far as I know has been fixed both Spigot side and minecraft 1.12.1 side.
dig some more digging on this and found @SupaHam is correct - Mojang has not fixed this yet. They did fix a security issue with the command, but that's all they did in 1.12.1 release related to this.
I'd like to see the recipe command back as well. We have a few custom recipes on my server that cannot show up in the vanilla recipe book, so being able to use /recipe to see them would be nice. Right now, we just have to have a page on our site listing them, and direct people to that.
I've found this plugin for the time being that seems to work pretty well. You have to have the recipe book disabled for regular players, though, to prevent them from being able to duplicate items.
https://dev.bukkit.org/projects/recipelookup
I've tested /recipe on Paper 1.13.2, and I can still reproduce the issue that caused the command to be disabled in the first place. /recipe won't be re-enabled until we've found a way to prevent the items being moved to the inventory.
In EssentialsX 2.16, you will be able to manually add force-enable-recipe: true to your config to prevent the safety check, but the exploit is still present and this option is not recommended.
Is there any possibility on using packets to send recipe lists instead of using real items?
Confirmed exploit still exists / is broken as of Essentials dev 713 and Paper dev 597 (Spigot 1.13.2) with it forced enabled.
This may be possible, but I don't feel it would be appropriate for EssentialsX to include any packet manipulation - this would require either tying us down to another external dependency, or writing our own packet code which would require significantly more maintenance effort.
I opened PaperMC/Paper#1783 a few months ago to see if there was any chance of adding API to Paper to allow us to cleanly fix this, but there hasn't been any movement on this so far.
@md678685 Bummer. :( I and my community liked the /recipe command as an in-game sort of recipe wiki. Now everyone just spams and asks how to craft stuff when they don't know something and are too lazy to look it up on the minecraft wiki.
I wonder if now that we're in 1.13, /recipe could be fixed? The command currently works, but no recipe can be found.
PaperMC/Paper#2107 adds an event that will let us enable /recipe on Paper, but we can't do this until it gets merged.
PaperMC/Paper#2107 (comment)
Can be /recipe restored?
PaperMC/Paper#2107 (comment)
Can be /recipe restored?
Maybe.
This is fixed for the latest versions of Paper for 1.15.2 and 1.16.1 respectively, on EssentialsX 2.17.2.143 or higher.
We do not provide support for the force-enable-recipe option. Use Paper if you want to re-enable /recipe.
The force-enable-recipe: true work around to reenable the command does not work on Spigot, EssentialsX 2.18
