ViaVersion / ViaRewind

ViaBackwards addon to allow 1.8.x and 1.7.x clients on newer server versions.
https://hangar.papermc.io/ViaVersion/ViaRewind
GNU General Public License v3.0
249 stars 79 forks source link

1.8 Client with Optifine invisible #314

Closed ludgart closed 3 years ago

ludgart commented 3 years ago

'/viaversion dump' Output

https://dump.viaversion.com/1aa16d93035d6187525956f4199e7dd01183324fc908c7a1aa963548594b938c

Console Error

No errors in the console

Bug Description

See "Steps to Reproduce"

Steps to Reproduce

Login with 1.8 with optifine installed (example forge or lunar) Join the 1.16.5 server with ViaVersion-4.0.0-21w19a, ViaRewind-2.0.0 and ViaBackwards-4.0.0-21w19a Die from a player Respawn NPC, Players and your self are invisible now (it look like that : https://media.discordapp.net/attachments/767315429189222450/843864694989258772/Screenshot_6.png?width=1791&height=875 ) Reconnecting doesnt help. Only a server restart help.

Expected Behavior

No visible bug

Additional Server Info

Proxy: Flamecore without any protocol plugin Server: Paper-1.16.5 - 629 Java 13 (and java 15 tested)

Filtered sent packets https://pastebin.com/BFsM2mBB It happen only for the player: Betman1234 ( bc of the client )

Checklist

kennytv commented 3 years ago

Sounds like it's because you use a 5 year old client version with old mods :p If you can reproduce it on a regular BungeeCord/Waterfall setup and without such rendering mods, we will reopen this (and move it to the ViaRewind tracker).

ludgart commented 3 years ago

Yea I know its the newest client version :D I reproduce it on a regular proxy setup without any mods in the forge client. Please move it to ViaRewind if they can help.

ludgart commented 3 years ago

Need to update my issue. It doesnt matter if forge or not. It happen to a lot of players. Example Lunar client. I think, because of optifine, because if we remove optifine from forge it work without problems.

Xernium commented 3 years ago

As you’ve undoubtedly recognized- Optifine is the issue here. This issue has been brought up multiple times and it’s not something we will look into further as it works with vanilla or unmodified clients.

ludgart commented 3 years ago

Wow, I'm disappointed. Why this project still exists? I switched to ViaVersion-3.2.1, ViaRewind-1.5.3 and ViaBackwards-3.2.0 and the bug is gone. But in this version is a other problem what I already reported it without success.

Xernium commented 3 years ago

As I said- ViaVersion is made to support Vanilla or unmodified clients. If modifications break that then we won’t help. That’s always been the deal, we can’t make it right for everyone. Besides that, 1.8 is quite a few years out of date now and rarely anyone still contributes code to maintaining these legacy versions. We chose to keep the older versions around because if we didn’t that would just spark another project to add them back. We compromised on not fixing bugs ourselves anymore associated with legacy versions more or less- But we are still taking in contributions by the community for maintenance of these versions.

ludgart commented 3 years ago

I'm pretty sure you know that the most Minecraft players use the "Mod" optifine. You guys have more experience to figure the problem out. I only can say it work with 3.2.1. There was a change what break it. I'm on your side, but in this case it breaks the gameplay immensely

Earthcomputer commented 3 years ago

Lol, imagine using quotation marks to call optifine a mod, it's a mod, and a pretty intrusive one at that, constantly breaking other mods.

ludgart commented 3 years ago

Oke, I found it. This version patch the invisible bug https://ci.viaversion.com/view/ViaBackwards/job/ViaBackwards-DEV/238/.

Plugin version list: ViaRewind-1.5.4-185 | vv Build-392 | vbackward Build-237 | work ViaRewind-1.5.4-185 | vv Build-392 | vbackward Build-238 | failed

Can someone look over this? Better would be a new branch with reverse of these changes to test it again.

kennytv commented 3 years ago

The commits from that build basically restore Vanilla behavior for respawns (which was previously broken for respawns in the same world for skin changers for example), so that's 100% on OptiFine - nothing we can fix, sorry. I'd suggest downgrading (or updating?) OptiFine to an earlier version or not using it.

ludgart commented 3 years ago

Wow, it took me hours to find the broken commits. It's not a solution for the players. A setting to restore the old behavior would be the best. But I see, reporting issues to improve the project is pointless. Second example https://github.com/ViaVersion/ViaRewind/issues/280. I think I need to create a fork of the project. You guys can "fix" it in 10 minutes and I need hours for it. Perfect.

Camotoy commented 3 years ago

Reporting issues to a project is not pointless. Reporting issues that only occur with clients that deviate from vanilla behavior is low priority or ignorable - if the mod breaks and not vanilla, it's probably the mod's fault, and not ViaVersion's. See if Optifine has an issue tracker you can contact.

ludgart commented 3 years ago

See #280 and a other issue, but i forgot the number

Reporting issues to a project is not pointless. Reporting issues that only occur with clients that deviate from vanilla behavior is low priority or ignorable - if the mod breaks and not vanilla, it's probably the mod's fault, and not ViaVersion's.

kennytv commented 3 years ago

^ and as I said, the commits restore Vanilla behavior and will otherwise break a good number of plugins... including the Vanilla client. Either you choose a different OptiFine version (which I am almost certain will fix it), or you have to report broken respawn packet handling to OF, assuming you are using its latest version. The other issue you linked is due to Vanilla restrictions of older clients, and PS works in some cases - in others it doesn't (also considering PS works completely differently to Via internally and is based on a different concept of packet transformation).

In any case, you should drop the attitude if you have no idea of the client's inner workings and also drop the expectation of version translation to an 8 year old version to work flawlessly in all possible scenarios - that's plain and simply unrealistic.