Minecraft Forge Server

[A Former User]

@murgero That does appear to be the case with the current release I have, at least from my testing

[nebulon]

I was just testing the app a bit and while I have no clue what the difference really is, it seems to mostly work

The logs and command interface in the app UI does not seem to work though as well as @atrilahiji what was the reason to put the actual server code (the .jars) into the git repo?

Ideally we would fetch those from some upstream repo with just the pinned version number, if possible.

[A Former User]

@nebulon I mean it’s pretty much a lightly modified version on your minecraft package. I’ll take a look at the console issues though.

For the jar files sitting in the package, this was initially because the install process requires running an installer jar file that uses a GUI to choose whether to install the client or server. I’m still trying to find a way to clean this up

[A Former User]

@nebulon I was thinking of hosting them somewhere else. But I’m checking to see what the process of building it from source looks like

[nebulon]

@atrilahiji well if there is a valid reason, then I don't think it is an issue to have the jar files in the repo.

[girish]

@atrilahiji Can you put something in the README on how you downloaded those jar files? That way, we can also know how to update it later.

[A Former User]

@girish yeah for sure. I’ll add that in a bit here

[A Former User]

@nebulon Perfect. I dont mind keeping them stored on a static file server too. But yeah the purpose of this is for mods compatible with Forge: https://www.curseforge.com/minecraft/mc-mods?filter-game-version=2020709689%3A7498&filter-sort=4

[nebulon]

Ah nice, I guess it would also be good if you could describe how to use such mods with this app package. I have only personally used the stock server thus far.

[A Former User]

@nebulon I added some quick guides. Might add have time to make a video or write a longer guide tomorrow. But I added some instructions in the README.

Also the frontend console issue is fixed and I bumped the forge version.

https://git.cloudron.io/AtriLahiji/minecraft-forge-app/-/releases/0.2.0

[nebulon]

Great, I think we are close to getting this into the app library. I've forked your repo into the cloudron namespace and you should be a "developer" for that repo at https://git.cloudron.io/cloudron/minecraft-forge-app

[A Former User]

@nebulon cool. Yeah the only thing I was thinking of changing there was hosting the actual server jar files and libs in a file storage server somewhere to reduce the bloat of the repo. Do you think that or the current method make the most sense?

[A Former User]

@brutalbirdie also perhaps this change I made to mine will help get the terminal working for yours? https://git.cloudron.io/cloudron/minecraft-forge-app/-/commit/b49f9b9ce3620dde5ca82aa4394345630120b251#d7eea7e71cfc09240e80c27cf7a3c1b6b21de82b

You’d have to see what the papermc server spits out

[BrutalBirdie]

@atrilahiji will try that.

[BrutalBirdie]

@atrilahiji That wont work for me
My Console output looks like this.

Feb 08 17:14:35 [16:14:35 INFO]: [Vault] No new version available
Feb 08 19:42:41 [18:42:41 INFO]: CONSOLE issued server command: /help
Feb 08 19:42:41 [18:42:41 INFO]: To view help from the console, type '?'.
Feb 08 19:42:55 [18:42:55 INFO]: -------------------- Help --------------------

so I would have to match \\[\d+\:\d+\:\d+\s[INFO|WARN|ERROR]\\]

[A Former User]

@brutalbirdie

[BrutalBirdie]

@atrilahiji I will test this

.toString().split(/\[\d+\:\d+\:\d+\s(INFO|WARN|ERROR)\]\:\s/)[2]);

Tested it with jsfiddle and regex101

text = "Feb 08 17:14:35 [16:14:35 INFO]: [Vault] No new version available";
console.log(text.toString().split(/\\[\d+\:\d+\:\d+\s(INFO|WARN|ERROR)\\]\:\s/)[2]);

[BrutalBirdie]

Well ... silly me did a copy-paste mistake and forgot the \ at the first [
also why not this RegEx \\[.*\\]\:\s a lot simpler.

[A Former User]

@brutalbirdie Lol true that should work

[A Former User]

@nebulon FYI I submitted a PR to the repo https://git.cloudron.io/cloudron/minecraft-forge-app/-/merge_requests/1

This should clean it up a bit more. Let me know if theres anything else you need from my end on getting this up and running as a proper app. Again, I am perfectly willing to be in charge of submitting MRs for any updates to take a bit of the load off of @staff