Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updade to 1.7.7 #18

Open
net32 opened this issue Apr 11, 2014 · 39 comments
Open

Updade to 1.7.7 #18

net32 opened this issue Apr 11, 2014 · 39 comments

Comments

@net32
Copy link

net32 commented Apr 11, 2014

hey, updade the project for compatibility with 1.7.8 😄

@Katelyn-SN
Copy link

That likely will never happen as that would requiring 3 protocols side by side when 2 is complicated enough already.

@literalplus
Copy link
Collaborator

Vanilla Bungee has it and supporting 1.7.2 and 1.7.(9 now) is just a few lines, if I remember correctly.
-- This message was sent from my tablet. Check my homepage!

On 15 Apr 2014, tf101-wizard [email protected] wrote:

That likely will never happen as that would requiring 3 protocols side
by side when 2 is complicated enough already.


Reply to this email directly or view it on GitHub:
#18 (comment)

@suchtie664
Copy link

Is it possible with this code?
SpigotMC@cd51869
and
SpigotMC@1d3adc5
Correct me if I wrong.

@literalplus
Copy link
Collaborator

Yes, that's what I was referring to.

(Might make a PR once I get time)

@Katelyn-SN
Copy link

You guys don't understand how it works you can't just drop these patches into this version of bungee as the protocol hack that this version uses is heavily modified nothing that you think will work with it will not. Since this system requires to run 2 different protocol systems side by side.

@literalplus
Copy link
Collaborator

I understand that it is not possible to throw in these patches 1:1, but I think you see that it is possible to support 1.7.9 too with little effort. (Or medium, if you want)
(For example, we'd need to have the uuid methods on players too)

@Katelyn-SN
Copy link

Its not as easy as you may think i did the patch already and it works and allows clients to connect but there is constant issues with it having to understand packets likely due to uuid name changes

@net32
Copy link
Author

net32 commented Apr 17, 2014

I did the code to adapt the UUID of version 1.7.6, but 1.6.4 Spigot is unsuitable for UUID when a player is in the same chunk that another conflict showing that the UUID is invalid UUID.

@Katelyn-SN
Copy link

Would you mind sharing the code so we can maybe work out a solution

@net32
Copy link
Author

net32 commented Apr 18, 2014

I will make a fork

@Sin96
Copy link

Sin96 commented Apr 18, 2014

i got a workin BungeeCord, which allows to join with 1.7.6+ on a 1.6.4 Spigot server. I have only one problem anymore. A 1.7.6+ Client doesn't load the skins.. so you cannot see a new spawned player for about 20-30 seconds. After that the client sets a standard skin. Do you have any ideas to fix that?

@Katelyn-SN
Copy link

Thats a very simple patch and was done by md_5 weeks ago SpigotMC@1d3adc5

Also please share your code we are eager to get our servers working as right now we are having to redirect users with direct ips

@Katelyn-SN
Copy link

Also @net32 the fork is not very helpful if you don't commit your changes to it :P

@net32
Copy link
Author

net32 commented Apr 19, 2014

@tf101-wizard hey, look at my fork: https://github.com/net32/BungeeCord
the error of a player being in the same chunk that other still persists only in version 1.7.6: http://prntscr.com/3beiv4

@Katelyn-SN
Copy link

Give me a minute mine worked all I did was change protocol mapping from 4-5
with no issues haha worst method but it worked

@Katelyn-SN
Copy link

Looks like you didn't isolate use of uuid for that client so basicly its looking for the uuid of the 1.6 player and cannot find it

@net32
Copy link
Author

net32 commented Apr 19, 2014

not yet understood why the error only occurs when two players are in the same chunk

@Katelyn-SN
Copy link

1.6.4 clients use the name system when 1.7.6+ clients use uuid support see if the error occurs with just changing the 4 to a 5 in packet mapping also send me your bungee and server logs so I can see why your uuid changes are causing that issue

@net32
Copy link
Author

net32 commented Apr 19, 2014

I'm almost giving up, I think I'll upgrade everything to 1.7 ):

@Katelyn-SN
Copy link

We can get it just get me those things I asked for

@net32
Copy link
Author

net32 commented Apr 19, 2014

make a clone of my fork and compile, the problem is only with the protocol 5, if you can get into my test server: 177.96.42.249

@Katelyn-SN
Copy link

No PC ATM u will have to do that on the long ride home from a small camping trip 1-2 hours I'll be around with a PC to help add my Skype kevhawk2914 so we can elaborate

@Sn0wStorm
Copy link

I have made a version that is fully working with the new Protocol, only drawback: Fake Players (NPCs) have no skin on Protocol 5. This is due to how the Skinsystem changed.

You are getting the Client Error, because you are not rewriting the Spawn Player Packet (The current rewrite just adds wowe_such_packet as UUID)

Its a rather hacky way to determine which protocol to rewrite the Spawn Player Packet to, but it works. I still have to clean up the code, remove Debug stuff, do testing and I have almost no time over the next days, so it may take a while.

@Sin96
Copy link

Sin96 commented Apr 19, 2014

can u upload it? maybe i can fix that spawn player problem

@Sn0wStorm
Copy link

Well as said its fully working except NPC Skins. Sadly I have no time to fork and upload it right now.

@suchtie664
Copy link

can somebody fix @net32 problem?

@Katelyn-SN
Copy link

Well that is essentially what Sn0wStorm is trying to fix can you possibly upload your source to dropbox @Sn0wStorm so i can get it done for these guys and if you can a diff :)

@net32
Copy link
Author

net32 commented Apr 20, 2014

@tf101-wizard @Sn0wStorm @suchtie664 @Sin96 : Solved: #20

@Sn0wStorm
Copy link

Sorry that I could not upload this earlier:

https://github.com/Sn0wStorm/BungeeCord

This is my fork of it, I will not create a Pull Request as of now, because the code ist not cleaned.

@Sn0wStorm
Copy link

There is still a Problem with Player Heads in 1.7.6 +
If a Player Head with Skin is within visible range, the Client crashes completely. Its no Packet related disconnect but a full crash. Does somebody know a solution to this problem? To reproduce you can give yourself a player head with skin with this command: /give name 397:3 1 player:name

@floh22
Copy link

floh22 commented Jun 26, 2014

well it seems like the fork @Sn0wStorm made still works with 1.7.10 though there weren't any major changes to its not very suprising ^^ Still very awesome fork :)

@net32
Copy link
Author

net32 commented Jun 26, 2014

yes, but still have the problem of head

@Koya07
Copy link

Koya07 commented Jul 4, 2014

Since I cannot speak English, I have translated.

only 1.7.7,it is made into the head of Steve.
I can not good programming.

@Sn0wStorm
Copy link

I am working on a solution for the Problem with the heads.

But it will not have any skin, so every head will be Steve for the new versions.

@Sn0wStorm
Copy link

Fixed here: Sn0wStorm@224b9ec

Skinned Heads will not show skins for the newer versions (1.7.10) much like NPCs.


If somebody wants to try to make skins show correctly on heads and NPCs, take a look at how Citizens2 fetches Skins here: https://github.com/CitizensDev/Citizens2/blob/master/src/main/java/net/citizensnpcs/npc/entity/HumanController.java
How the Skin NBT-Data has changed: Bottom of the Page here, under Skull, the whole Owner Tag-Compound would need to be created: http://minecraft.gamepedia.com/Chunk_format#Tile_Entity_Format

@Koya07
Copy link

Koya07 commented Jul 10, 2014

If https://github.com/Sn0 wStorm/BungeeCord is Build and it tries to go into a server, an error will come out.

io.netty.util.concurrent.SingleThreadEventExecutor runAllTasks
Warning: A task raised an exception.
java.lang.NullPointerException
at net.md_5.bungee.connection.InitialHandler$8$1.run(InitialHandler.java:513)
at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:354)
at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:353)
at io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:101)
at java.lang.Thread.run(Unknown Source)

@Koya07
Copy link

Koya07 commented Jul 14, 2014

An error will be made if Offlinemode=false is used.

io.netty.util.concurrent.SingleThreadEventExecutor runAllTasks
Warning: A task raised an exception.
java.lang.NullPointerException
at net.md_5.bungee.connection.InitialHandler$8$1.run(InitialHandler.java:513)
at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:354)
at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:353)
at io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:101)
at java.lang.Thread.run(Unknown Source)

@Sn0wStorm
Copy link

Hm, i did not test offline-mode

@Koya07
Copy link

Koya07 commented Jul 15, 2014

I want to add, if possible the offline mode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants