chirs241097 / jline4mcdsrv

Command history, auto completion and syntax highlighting for every fabric server.
MIT License
24 stars 4 forks source link

[Suggestion] Load earlier for better reading some issues / Compat with Jline #27

Open suerion opened 5 days ago

suerion commented 5 days ago

Hey, could you load the mod earlier to look more lines with "jline4"?

image

it is starting the mod on line 6k console log, but i think it could be start earlier.

Could you look on this? Love your mod, because it works fine with all other mods

And please add Compat with this mod -> https://modrinth.com/mod/better-fabric-console

i Love both mods

Fourmisain commented 5 days ago

I believe loading jline4mcdsrv any earlier will break compatibility with Better Log4j Config (#25) and Not Enough Crashes (<4.3.0) / StackDeobfuscator (#5).

I'm also not sure if it's possible to load it early enough so that all logs are formatted correctly because it would need to precede Fabric Loader itself...?

The way to fix this (at least the way I do) is to use a custom Log4j2 config with the same log pattern as jline4mcdsrv and starting your server with it. You can find a working Log4j2 config with jline4mcdsrv default pattern here. You just have to add its path as a JVM arg: -Dlog4j2.configurationFile=file:///D:/Path/To/Your/log4j2.xml

suerion commented 5 days ago

Oh yes, i think i will look on it :) i try to get the best logging way to work on my dev enviroment, to help others with bug fixes :) thanks for this