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

[Feature] Support GeyserMC if possible (Edit Watchdog: Minecraft Bedrock Clients connect to Java Server) #70

Closed
Cameronsplaze opened this issue Nov 14, 2024 · 3 comments · Fixed by #74
Labels
enhancement New feature or request

Comments

@Cameronsplaze
Copy link
Owner

Is your feature request related to a problem? Please describe.
Just learned GeyserMC exists, which lets bedrock CLIENTS connect to a java SERVER. It'd be fun to see what it takes to support this, but according to this issue, you might need BOTH tcp and udp logic for java and bedrock respectively.

This could mean a re-write of the watchdog logic, so I'd like to wait until #21 is figured out. It could mean if that works out with just metrics, we solve this issue for free.

I think if possible, I'll just add this to the minecraft-java config with clear comments on what to delete if you don't want it. I thought about maintaining a third config specifically for this, but there'd still be the simple bedrock version anyways. Every config adds considerable time to synth, deploy, and just maintain in general anyways.

@Cameronsplaze Cameronsplaze added the enhancement New feature or request label Nov 14, 2024
@Cameronsplaze
Copy link
Owner Author

Cameronsplaze commented Nov 18, 2024

Poked at this a little bit. Was able to connect to the server on a mobile device!

But it only lasted 5 minutes, i.e the watchdog couldn't see the bedrock server connections. Technically I can re-write the watchdog logic to support it watching BOTH tcp and udp at the same time, but there's no point doing that much work unless #21 doesn't pan out.

I guess to get off the ground, I could just add a third type to tcp and udp, that just looks at the amount of traffic going from the container or something

@Cameronsplaze Cameronsplaze linked a pull request Nov 24, 2024 that will close this issue
8 tasks
@j1mmie
Copy link

j1mmie commented Dec 31, 2024

Hi @Cameronsplaze, sorry to ping you, I'm having trouble following the history of this topic and the various possible solutions. I set up a new Minecraft OnDemand server (actually PaperMC, with Geyser) last week and have also been experiencing this issue - my server shuts down 10 minutes after starting up, despite having a Bedrock client connected. Is this expected to be fixed yet?

@Cameronsplaze
Copy link
Owner Author

Hi @j1mmie, no worries! Especially since this issue is closed, I might not see any reply unless you ping. Keep it up just in case. I didn't think anyone was using the repo yet, so most of the issues are more "personal notes" lol. I'm surprised bedrock would spin it down though, those tend to be LOADS more noisy than java clients. What I'd suggest is:

  • Pull from main and re-deploy, I recently fixed something that makes threshold more stable than it used to be.
  • Grab the updated value(s) from ./Examples/Minecraft.java.example.yaml and update your config. Especially the new Watchdog variables.
  • Watch the dashboard, and see what the traffic looks like for your specific use-case (More info on this here).

Of note though, my next project is moving the hosted zone logic into the leaf stack. I didn't realize there was a restriction on filters, so right now you can only have two leaf stacks per account. I'm not sure if cdk deploy is smart enough to update everyone automatically once I do, or if you'll have to destroy/re-deploy everything later. That's the last "breaking change" I currently see for this project, but also why I haven't advertised it yet.

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

Successfully merging a pull request may close this issue.

2 participants