Armory Expansion

Armory Expansion

10M Downloads

Server Crash on Forge 2835 with 1.1.10

TwilightPSN opened this issue · 38 comments

commented

Server is currently crashing and wont boot with the newest update

[minecraft/MinecraftServer]: Encountered an unexpected exception 13.04 09:13:59 [Server] INFO net.minecraftforge.fml.common.LoaderExceptionModCrash: Caught exception from Armory Expansion - Construct's Armory (armoryexpansion-conarm) 13.04 09:13:59 [Server] INFO Caused by: java.lang.NullPointerException 13.04 09:13:59 [Server] INFO at java.util.Objects.requireNonNull(Objects.java:203) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at org.softc.armoryexpansion.integration.ConArmIntegration.loadMaterialsFromOtherIntegrations(ConArmIntegration.java:61) ~[ConArmIntegration.class:?] 13.04 09:13:59 [Server] INFO at org.softc.armoryexpansion.integration.ConArmIntegration.preInit(ConArmIntegration.java:73) ~[ConArmIntegration.class:?] 13.04 09:13:59 [Server] INFO at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at net.minecraftforge.fml.common.FMLModContainer.handleModStateEvent(FMLModContainer.java:637) ~[custom256.jar:?] 13.04 09:13:59 [Server] INFO at sun.reflect.GeneratedMethodAccessor9.invoke(Unknown Source) ~[?:?] 13.04 09:13:59 [Server] INFO at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_201] 13.04 09:13:59 [Server] INFO at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:91) ~[minecraft_server.1.12.2.jar:?] 13.04 09:13:59 [Server] INFO at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:150) ~[minecraft_server.1.12.2.jar:?] 13.04 09:13:59 [Server] INFO at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:76) ~[minecraft_server.1.12.2.jar:?] 13.04 09:13:59 [Server] INFO at com.google.common.util.concurrent.MoreExecutors$DirectExecutor.execute(MoreExecutors.java:399) ~[minecraft_server.1.12.2.jar:?]

commented

oh nothing at all i just stay on 2815 as i havent had mods need anything higher

commented

You mentioned the crash was on 2835 though ^^''

commented

as well as 2815 i reverted back to 2815 as 2835 broke my server

commented

I fixed that crash but ended up with another one... It might take be a bit to figure it out entirely. Sorry

commented

Have you tried deleting the old config files?

commented

yes removed the configs and rebooted i also returned server to previous 2815 i was on and same issue

commented

it doesn't seem to be a forge issue, can I have you roll back and see if it still happens? meanwhile I'll look for a fix

commented

1.1.7b is the version that works for me

commented

Nothing past that one works? That's odd, I myself am running a dedicated server with v1.1.10 and I know of others running servers with v1.1.9b for example. My server is running forge 2821 and my client as well.

commented

actually let me try 1.1.10 again i just notice theres a folder config as well for the mod and didnt see it before will report back momentarily

commented

okay after testing each version im only able to get 1.1.7b running i removed configs with every change and still no luck. even tried removing the constructs armory configs

commented

here is the modlist of what im running if that helps
https://www.modlister.com/user/TwilightPSN

commented

is there anything stopping you from using forge 2821?

commented

seems to be related to this https://pastebin.com/vDw9GJ4q
are you using a linux server?
a linux server user told me about this just now

commented

Unbenannt2
it seems to grab and generate some / in the configs directory so a linux server cant run them and serachs ofor them but cant find it

commented

I am also not able to run anything higher than 1.1.7b on my server.
crash-2019-04-14_19.58.45-server.txt

Here I've updated my forge version and confirmed that comment above is correct.
These files are placed in the server's root folder instead of infact in the config file.
crash-2019-04-14_20.36.35-server.txt

commented

Thanks to all the info it seems I might be able to solve this quite soon

commented

Hi! I'm in 1.12.2 and Forge version 2825, and I'm getting this crash on my server (singleplayer seems fine): https://hastebin.com/idifudaroy.sql

Do you think it's the same issue, or should I make a new ticket?

commented

@katubug and @kreezxil are the two of you running on a linux machine?

commented

confirmed:

1.1.10 crashes dedicated server Forge 1.12.2 build 2825

Notes:
npe

crash report:
http://paste.ubuntu.com/p/9YPrwwBqcv/

fml server latest:
http://paste.ubuntu.com/p/Gb2FDDgP8X/

commented

I though it might be the fact I was using \ instead of / but a test build with that change still crashed on Linux :/

commented

I host with bisecthosting.com, and I don't see where they list server specs.

commented

I host on apexminecrafthosting.com no idea their servers

commented

I'm using linux to host my server and I suffer this crash with files like config\armoryexpansion... , just like SirAron111.
I also make a test server on windows 10, and everything seems fine.

commented
commented

Was experiencing the same crash on my Linux desktop, running OpenJDK 8 on Arch Linux 4.14.68-1-lts. I ran a blanket replace on all instances of \\ to / and rebuilt, it seems to be working on my end!

commented

Yes, I use Linux to host all my servers.

On Mon, Apr 15, 2019 at 5:24 PM Bruno Miguel Faria Gaspar < @.***> wrote: I though it might be the fact I was using \ instead of / but a test build with that change still crashed on Linux :/ — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#33 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AA5TJCAH5V3HB3R46BS3MWLPQT6DXANCNFSM4HFYFJVQ .

99% of server hosts that I've ever used/tried use Linux for their servers. o.o
In that regard, this is a big deal to have not working properly for servers.

commented

@lamialily @zerafox @LemADEC I made a test build, replacing all \ with / but crashes still ocurred, nonetheless, I'll do some more digging around in hopes of having a fix by this week.

commented

please push last changes to github, see if we can help

commented

@YaibaToKen You made sure to replace it in both ConArmIntegration.java and CustomMaterialsIntegration.java, right?

commented

yup, I believe so, but lemme push the latest changes. Any and all PR's are appreciated btw

commented

@lamialily If you can mod you could try editing the file and see if replacing them in ConArmIntegration.java and CustomMaterialsIntegration.java lets it start on Linux if that works for you it would be good. just make a pull request afterwards if you find the fix I bet that would help. O I was to slow XD he already answered

commented

@SirAron111 I did say I rebuilt and it worked :P But yeah, I'll see about merging and push a PR once the latest changes are up.

commented

Committed and pushed

commented

You forgot in ConArmIntegration.java at line 58

commented

You never want to use "" for paths in your Java programs.
For example, here: https://github.com/YaibaToKen/Armory-Expansion/blob/b69f5f171f969aa2b0cef954c95e267b4ed46bce/src/main/java/org/softc/armoryexpansion/integration/CustomMaterialsIntegration.java#L67

        File output = new File(configDir.getPath() + "\\armoryexpansion\\traits.txt");

Consider always using '/' which both Windows and Unix can support, or use a intermediary step on the folder itself, like so:

	// create mod folder
	fileConfigDirectory = new File(configDir.getPath());
	//noinspection ResultOfMethodCallIgnored
	fileConfigDirectory.mkdir();
	if (!fileConfigDirectory.isDirectory()) {
		throw new RuntimeException(String.format("Unable to create config directory %s",
		                                         configDir.getPath()));
	}
	// open traits configuration file
	File output = new File(fileConfigDirectory, "traits.txt")
commented

You've got my PR, at least :D #35

commented

Ah crud :X I'm not home atm but I'll push the mentioned change asap. Hopefully that'll be the end of it.