shutter-network / rolling-shutter

Rolling Shutter is an MEV protection system to be plugged into rollups.
https://twitter.com/project_shutter/
28 stars 9 forks source link

Fix config parsing defaults #399

Closed ezdac closed 1 year ago

ezdac commented 1 year ago

The configuration parser did exclude the wrong set of configuration parameters during setting the default values. The cause for this was a bug / unexpected behaviour in viper.AllKeys() - the function returned also keys that had a nil value set internally. Those do not represent values that were set by the user, and thus should not be excluded from the default value override.