haveachin/infrared

Seems like an error with config refresh

realspinelle opened this issue · 4 comments

So its really easy to reproduce default java config file

java:
gateways:
default:
listeners:
default:
bind: :25566
servers:
default:
gateways:
- default
domains:
- "*"
address: 192.168.10.104:25565
overrideStatus:
versionName: Infrared
maxPlayerCount: 777777777
playerCount: 2

so versionName is gonna be 'Infrared' right ? yes it is

but now remove the whole overrideStatus section is the versionName '1.12.2' (server version) ? no its Infrared

Could you please post screenshots of what you are referring to? I have a feeling that you might be looking at the dialTimeoutStatus by accident.

ok so if i start the server with that conf
java:
gateways:
default:
listeners:
default:
bind: :25566
servers:
default:
gateways:

  • default
    domains:
  • "*"
    address: 192.168.10.104:25565

image

then i edit it to
java:
gateways:
default:
listeners:
default:
bind: :25566
servers:
default:
gateways:

  • default
    domains:
  • "*"
    address: 192.168.10.104:25565
    overrideStatus:
    versionName: Infrared
    maxPlayerCount: 777777777
    playerCount: 2

image

then i go back to
java:
gateways:
default:
listeners:
default:
bind: :25566
servers:
default:
gateways:

  • default
    domains:
  • "*"
    address: 192.168.10.104:25565

image

if i reboot its returning back to 1.12.2 but should not happen right ? x)

I think I found the issue. It has something to do with the library I use for merging the config maps. I have absolutely no clue why it behaves that way, but I seems like it caches the pointer to that string. I will have to investigate that further and probably end up opening an issue or writing a map merge function myself.