docker_factorio_server icon indicating copy to clipboard operation
docker_factorio_server copied to clipboard

Published 20 hours ago • verified publisher icon zopanix

Unable to run latest builds

Open bkuhl opened this issue 8 years ago • 10 comments

We're unable to run the 0.14 build. --latency-ms is no longer an option for the client given the rework of networking on Factorio. It looks like server-settings.json format may have changed as the smart launch is not building a usable file leading to the error Error util.cpp:57: /opt/factorio/server-settings.json(22): expected value when run.

bkuhl avatar Sep 20 '16 22:09 bkuhl

It seems to not like the verify_user_identity being set by the variable. I didn't dig too much, but if I remove the variable and just put in false, it seems to work. I basically updated the config in the script with the latest, removed the --latency-ms stuff and hard set the verify_user_identity and everything is working. I would submit a pull request but the hard setting the verify_user_identity isn't ideal, but I thought I would share so that someone more qualified might take a crack at it. o7

duckman avatar Sep 26 '16 15:09 duckman

I've disabled the automatic releasing of new versions until this repo is compatible with 0.14.

bkuhl avatar Oct 01 '16 21:10 bkuhl

Is the linked PR going to be the fix moving forward, then?

pokemane avatar Oct 02 '16 18:10 pokemane

No, not fully. I think Duckman needs to submit a PR

bkuhl avatar Oct 03 '16 03:10 bkuhl

@duckman Can you please submit a PR with your changes?

bkuhl avatar Oct 09 '16 21:10 bkuhl

Are you sure it doesn't just need quotes around it? :P

FreekingDean avatar Oct 15 '16 19:10 FreekingDean

Yes, I'm sure.

bkuhl avatar Oct 16 '16 18:10 bkuhl

It seems, that by putting quotes around $FACTORIO_SERVER_VERIFY_IDENTITY in the config file template along with removing

--allow-commands --autosave-interval --autosave-slots

0.14.14 works!

I would submit a PR, but I don't think that just removing those commands is the best fix. It is fine for my needs, but it should be investigated if it is possible to add to the config file.

FreekingDean avatar Oct 18 '16 06:10 FreekingDean

I gave up on command line switches and ENV and mounted server-config.json into the /opt/factorio/data folder. It ended up being simpler. On the other hand, you have to copy up the config file. I suppose it's a matter of taste.

https://github.com/dtandersen/docker_factorio_server/

dtandersen avatar Oct 25 '16 19:10 dtandersen

Will this issue be resolved in the near future? just pulled the image and got the same error :(

niklas-dahl avatar Dec 16 '16 20:12 niklas-dahl