Development has effectively stopped on the 1.2.x branch and there's no
interest in keeping it going. Put up a warning that this version is no
longer supported and direct people to go to main clusterio project page
instead.
[skip ci]
The Travis integration is completely broken for 1.2.x due to some issue
with the npm cache and the linked packages created in it by the master
branch. Remove it from here.
In 1bdf23d I incorrectly assumed that the response sent wouldn't matter
as it hadn't sent a response in ages. That was not the case and the
client code checks to see if the response is "ok" while .sendStatus(200)
sends a response of "OK". Fix log spam by doing a case insensitive
comparison of the response.
According to rfc2616#section-6:
> After receiving and interpreting a request message, a server
> responds with an HTTP response message.
That response was missing for /api/getID which could lead to
EMFILE, because needle module was waiting indefiniately for
response from server. Since client sends /api/getID once every 10
seconds, this piles up, ultimately reaching ulimit for opened file
descriptors.
Signed-off-by: Michał Łyszczek <michal.lyszczek@bofc.pl>
Remove the obsolete and broken statistics gather from the clusterioMod
plugin. The statisticsExporter plugin already exports all of the
statistics (and more) that was exported here.
When updating a mod Hotpatch updates the loaded_mods array but not the
global.mods array. This causes events to be sent in a different order
on the client/server. Fix by updating global.mods in install_mod.
Fix the inventoryImports client side code never receiving the script
output by specifying fileReadDelay and thereby getting the old
behaviour. This broke with the introduction of the tail mode script
output mode which requires a newline to be written at the end of
the output.
Fix the inventory imports command trying to count up the removed quick
bar slots as well as using player_main instead of character_main.
Add warning that the lua commands file containing the unminified vesion
of the command has not been kept up to date. There are multiple changes
that has been done previously so these have already diverged.
The default limit of 100kb to JSON payloads of bodyParser can be
exceeded when running modded games with researchSync. Increase the
limit to 10mb as a bandaid to fix this issue.
Set uploadModsToMaster to false in config.json.dist. This feature is
not necessary as the integrated mod portal in Factorio provides a better
method to synchronize mods.
Update instructions, Dockerfile and Travis CI to use Node.js 10.
Specify that Node.js version 10 or greater is needed in the engines
section of package.json (note that this is only advisory.) And document
this as a breaking change in CHANGELOG.md.
This is a backport of 69a1ff7.
Reorder the install pm2 step to be after the npm install step in the
instructions. This is necessary to avoid npm creating some files in the
home directory owned as root and then failing the npm install step
because of this.
Reword the warning at the start of the readme to be less intimidating,
as this is the correct version of the project that users are expected to
install.
Fix the next level of an infinite research getting updated with the
progress of the previous level from other nodes in the cluster by
filtering out infinite researches with a lower level than the local
level.
Set the progress for the next level of an infinite research when
unlocking it via a script. This is necessary when the research is
actively being researched as Factorio will carry over the progress for
the current level into the next one when set via a script.
It's possible for the new progress to be non-zero if the next level was
on the research queue.
Fix naive checks for properties being set of the form obj[key] into the
proper form Object.hasOwnProperty.call(obj, key). The reason for this
is that if the key has a value of a built-in property like 'constructor'
then a check of the form obj[key] will always return true even if that
property hasn't been set on the object.
Hornwitser