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

[1.13.2] Getting disconnected #31

Closed
chaos234 opened this issue Apr 3, 2019 · 9 comments
Closed

[1.13.2] Getting disconnected #31

chaos234 opened this issue Apr 3, 2019 · 9 comments
Labels

Comments

@chaos234
Copy link

chaos234 commented Apr 3, 2019

I use on my server and on my client the same AppleSkin Versions but getting kicked from the server: https://i.imgur.com/f0o5WfL.png

Why is there such check and can I disable it because such checks are sucking hard. I understand them only, if the mod changes the gameplay but this mod doesn't do it so please remove this check or implement an option to disable it, because it isn't needed, thanks.

@squeek502
Copy link
Owner

Not sure, will look into it when I get a chance. What Forge version are you using?

(note to myself) Relevant code: https://github.com/squeek502/AppleSkin/blob/1.13.2/java/squeek/appleskin/network/SyncHandler.java#L24-L26

@squeek502 squeek502 added the bug label Apr 3, 2019
@squeek502 squeek502 changed the title Getting disconnected [1.13.2] Getting disconnected Apr 3, 2019
@chaos234
Copy link
Author

chaos234 commented Apr 4, 2019

Forge: 1.13.2-25.0.107

@TheLimePixel
Copy link

I was about to report it, but I see it already is reported. However, I don't see the actual error because of which the player gets kicked in your image, so here it is https://pastebin.com/R1GCyAtd

@squeek502
Copy link
Owner

squeek502 commented Apr 6, 2019

I was about to report it, but I see it already is reported. However, I don't see the actual error because of which the player gets kicked in your image, so here it is https://pastebin.com/R1GCyAtd

Is there more of the error below that? If so, could you post that too?

squeek502 added a commit that referenced this issue Apr 7, 2019
@squeek502
Copy link
Owner

@TheLimePixel
Copy link

Yeah, that change caused #32

@squeek502
Copy link
Owner

Yeah, that change caused #32

#32 is a separate problem caused by a change in Forge.

@TheLimePixel
Copy link

🤦‍♂️ I forgot I updated forge... derp

@squeek502
Copy link
Owner

Closing assuming this is fixed. @chaos234 please re-open if v1.0.10 didn't fix it for you.

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

No branches or pull requests

3 participants