The config is not thread safe, the Config.CurrentMagicBlock , Config.Miners, Config.Sharders, etc all are being updated and read concurrently with out lock protecting.
We should separate the data that will be updated periodically from the Config struct. Config should only hold static configuration data that is loaded on starts.
The config is not thread safe, the
Config.CurrentMagicBlock
,Config.Miners
,Config.Sharders
, etc all are being updated and read concurrently with out lock protecting.We should separate the data that will be updated periodically from the
Config
struct.Config
should only hold static configuration data that is loaded on starts.