pandorabox-io / pandorabox.io

Pandorabox infrastructure code
https://pandorabox.io
31 stars 4 forks source link

Woodcutting Crash (again...) #443

Closed BobFred7 closed 4 years ago

BobFred7 commented 4 years ago
2020-02-12 02:53:52: ERROR[Main]: ServerError: AsyncErr: environment_Step: Runtime error from mod 'woodcutting' in callback environment_Step(): /data/world//worldmods/woodcutting/init.lua:124: attempt to concatenate a nil value
2020-02-12 02:53:52: ERROR[Main]: stack traceback:
2020-02-12 02:53:52: ERROR[Main]:   /data/world//worldmods/woodcutting/init.lua:124: in function 'get_delay_time'
2020-02-12 02:53:52: ERROR[Main]:   /data/world//worldmods/woodcutting/init.lua:188: in function 'func'
2020-02-12 02:53:52: ERROR[Main]:   /usr/local/share/minetest/builtin/common/after.lua:20: in function 'globalstep'
2020-02-12 02:53:52: ERROR[Main]:   ...//worldmods/monitoring/monitoring/builtin/globalstep.lua:29: in function <...//worldmods/monitoring/monitoring/builtin/globalstep.lua:20>
2020-02-12 02:53:52: ERROR[Main]:   /usr/local/share/minetest/builtin/game/register.lua:429: in function </usr/local/share/minetest/builtin/game/register.lua:413>
2020-02-12 02:53:52: ERROR[Main]: stack traceback:
BobFred7 commented 4 years ago

https://github.com/pandorabox-io/pandorabox.io/issues/360 Seems to be the same as this

thomasrudin commented 4 years ago

Thanks for the report :+1: , i think that justifies a fork and proper nil-check for that condition...

BobFred7 commented 4 years ago

It happened yet again at 2020-02-13 01:08:26

thomasrudin commented 4 years ago

i think that justifies a fork and proper nil-check for that condition...

Heh, we already use a fork, but the error-reporting mechanism threw an error :stuck_out_tongue:

Looks like SwissalpS made a PR for this last year and i somehow did not see that... :confounded:

https://github.com/thomasrudin-mt/woodcutting/pull/1

Closing, the next time this happens a proper error-message should appear in the logs...