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

Improve transfer speed & Fix an issue that marks the mod useless #1

Open
wants to merge 2 commits into
base: 1.18
Choose a base branch
from

Conversation

msparkles
Copy link

As titled.

@msparkles
Copy link
Author

New commit: Change hostIp to be externalIp which is the public ip of the host, and set it to listen on 0.0.0.0

I don't know how exactly internet works, but at least from my observation, the mod doesn't work for players not in the local network currently. The commit aims to fix that by asking to fill in the public IP.

@msparkles msparkles changed the title Use BufferedInputStream#transferTo to improve transfer speed Use BufferedInputStream#transferTo to improve transfer speed & Change hostIp to be externalIp which is the public ip of the host, and set it to listen on 0.0.0.0 Dec 29, 2021
@msparkles msparkles changed the title Use BufferedInputStream#transferTo to improve transfer speed & Change hostIp to be externalIp which is the public ip of the host, and set it to listen on 0.0.0.0 Improve transfer speed & Fix an issue that marks the mod useless Feb 27, 2022
@msparkles
Copy link
Author

I need to learn how to use Git.

@msparkles
Copy link
Author

Also fixes a bug where it hangs the server up after typing /stop
The issue is due to unclosed HTTP server.

…alIp which is the public ip of the host, and set it to listen on 0.0.0.0, Use BufferedInputStream#transferTo to improve transfer speed
@tomgie
Copy link

tomgie commented Mar 19, 2022

Is this ever going to be merged?

@msparkles
Copy link
Author

Who knows, better to use halotroop's fork of my fork.

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

Successfully merging this pull request may close these issues.

2 participants