docker-palworld-dedicated-server icon indicating copy to clipboard operation
docker-palworld-dedicated-server copied to clipboard

Enable/Disable RCON console via env variable

Open Lexiv opened this issue 1 year ago • 2 comments
trafficstars

Lexiv avatar Jan 22 '24 11:01 Lexiv

Sorry @Lexiv i have already implemented a solution to this with 3 other fixes and testing them right now.

jammsen avatar Jan 22 '24 12:01 jammsen

Hey @Lexiv Your pull-request should be already included in this https://github.com/jammsen/docker-palworld-dedicated-server/commit/ee23b8807b280d2787af71ab18cfb7a0cc51b577

Feel free to review it and decide if you miss something and let me please know

jammsen avatar Jan 22 '24 14:01 jammsen

@Lexiv ?

jammsen avatar Jan 23 '24 10:01 jammsen

Sorry! I must have missed that somehow. Looks good.

I think you don’t need to be able to edit the games RCON port since this can be mapped to any port on the outside.

Lexiv avatar Jan 23 '24 10:01 Lexiv

Sorry! I must have missed that somehow. Looks good.

No problem 👍

I think you don’t need to be able to edit the games RCON port since this can be mapped to any port on the outside.

Ohhh ... hmmm ... you bring up a good point ... we dont give the ability to set the PalServer.sh "port=1234" either ... changing this would be consistent with the other change right? @Lexiv

jammsen avatar Jan 23 '24 11:01 jammsen

Sorry! I must have missed that somehow. Looks good.

No problem 👍

I think you don’t need to be able to edit the games RCON port since this can be mapped to any port on the outside.

Ohhh ... hmmm ... you bring up a good point ... we dont give the ability to set the PalServer.sh "port=1234" either ... changing this would be consistent with the other change right? @Lexiv

Yeah because that wouldn’t really make sense either since it can be mapped to any port on the host

Lexiv avatar Jan 23 '24 14:01 Lexiv

Yeah and the Dockerfile defintion is hardcoded anyway. Thanks for your input! Do you want to close this PR? Im gonna fix the rest directly.

jammsen avatar Jan 23 '24 16:01 jammsen