aergoio / aergo

aergo blockchain kernel
http://docs.aergo.io
MIT License
214 stars 45 forks source link

disable some lua functions #306

Closed kroggen closed 5 months ago

kroggen commented 1 year ago

This PR removes some Lua functions, starting on hardfork 4:

They continue existing for previous hardforks for compatibility with gas computations. The first 4 are also available when Aergo is build in debug mode

This PR also does not build some disabled packages (os, io, jit, debug) and so the code cannot be accessed in any way

It includes tests to make sure the disabled modules and functions are not available

kslee8282 commented 12 months ago

sync success

kslee8282 commented 12 months ago

Nov 1 05:18:42.040 INF ../go/aergo/cmd/aergosvr/aergosvr.go:100 > AERGO SVR STARTED branch=topic/disable-lua-require module=asvr revision=b9d3bb50

kroggen commented 12 months ago

@kslee8282 OK, but a new sync test will be needed later. But probably not just for this PR. We can merge many into one branch to make a sync test for V4 later