Enigmatica6 icon indicating copy to clipboard operation
Enigmatica6 copied to clipboard

Major Infinite Fortune exploit

Open Butternubbs opened this issue 2 years ago • 13 comments

Modpack Version

1.4.1

Describe your issue.

Apotheosis anvils function strangely with the 'Consistent Luck' Astral Sorcery perk: When holding the anvil, an extra level of Fortune is applied to it, and that enchantment holds when placing the anvil down. If mined and repeated, the anvil's Fortune enchant can stack infinitely. Moving the enchant to a book with the Pressure Chamber allows the player to obtain an infinite Fortune pickaxe, etc. Also works with other enchants such as Unbreaking, Sweeping Edge, etc.

Crash Report

No response

Latest Log

https://gist.github.com/Butternubbs/474e7867c51a3bcc4d2b89909d54cf94

Have you modified the modpack?

No

User Modifications

No response

Did the issue happen in singleplayer or on a server?

Singleplayer

Discord Username

Butternubbs#9827

Butternubbs avatar Aug 15 '22 03:08 Butternubbs

Thanks for the report - this was meant for Enigmatica 6 right? :)

NielsPilgaard avatar Aug 15 '22 05:08 NielsPilgaard

Oops, yeah for 6. Went to the wrong GitHub! 😅

On Mon, Aug 15, 2022 at 12:37 AM Niels Pilgaard Grøndahl < @.***> wrote:

Thanks for the report - this was meant for Enigmatica 6 right? :)

— Reply to this email directly, view it on GitHub https://github.com/EnigmaticaModpacks/Enigmatica6/issues/5167, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJF2SW74ME7MMDXIE27QUATVZHJRVANCNFSM56Q7J6BQ . You are receiving this because you authored the thread.Message ID: @.***>

Butternubbs avatar Aug 15 '22 05:08 Butternubbs

Hmm.. I guess it's actually working as intended :P I think the only solution would be to blacklist anvils, if that's possible. Edit: Not possible 😅

NielsPilgaard avatar Aug 15 '22 08:08 NielsPilgaard

I thought we could disable specific perks?

MuteTiefling avatar Aug 15 '22 13:08 MuteTiefling

I might've missed it, but didn't see it in either config :/

NielsPilgaard avatar Aug 15 '22 19:08 NielsPilgaard

Looks like Shadows can fix this more cleanly from their end. They'll be releasing a fix that'll remove invalid enchants from anvils on placement.

MuteTiefling avatar Aug 16 '22 03:08 MuteTiefling

Apotheosis-1.16.5-4.8.9A0 fixes this

MuteTiefling avatar Aug 16 '22 15:08 MuteTiefling

I'm sorry it took so long to comment this, I didn't have internet for the last 30 hours. You can do the same enchant dupe with pnc drones.

Vashal0 avatar Aug 18 '22 20:08 Vashal0

Not sure how that works since drones don't get your perks.

Regardless, the patched apotheosis strips invalid enchants when the anvil is placed. Since fortune isn't valid for anvils, it just gets removed.

MuteTiefling avatar Aug 18 '22 22:08 MuteTiefling

Drones are enchantable, and so they get the Astral perk enchants when held in your hand the same way an anvil does. When put down they keep the enchant just like the anvils did

Vashal0 avatar Aug 19 '22 00:08 Vashal0

Ah, that makes more sense. Will get that reported over to PNC shortly then.

MuteTiefling avatar Aug 19 '22 00:08 MuteTiefling

Sounds like PNC needs to apply the same fix - bleh. That change is here, for reference https://github.com/Shadows-of-Fire/Apotheosis/commit/d80db9c912d976161f0366b758ef384d0bb8004c#diff-935f61a18acf17035df523bcde818dc5a090249192234993045ba5d5a63a2495R91

Shadows-of-Fire avatar Aug 19 '22 03:08 Shadows-of-Fire

Specifically they'll have to do the filtering at this line https://github.com/TeamPneumatic/pnc-repressurized/blob/c792edc19674b4f3b85fd3b95247c3b6683be67c/src/main/java/me/desht/pneumaticcraft/common/entity/living/EntityDrone.java#L283

GitHub
A port of PneumaticCraft to MC1.12+. Contribute to TeamPneumatic/pnc-repressurized development by creating an account on GitHub.

Shadows-of-Fire avatar Aug 19 '22 03:08 Shadows-of-Fire