Commit 9c4ff1e7 authored by Seungju Lee's avatar Seungju Lee

fix: check op-batcher config flags when instantiating

parent d3c3a230
...@@ -27,6 +27,9 @@ func Main(version string, cliCtx *cli.Context) error { ...@@ -27,6 +27,9 @@ func Main(version string, cliCtx *cli.Context) error {
return err return err
} }
cfg := NewConfig(cliCtx) cfg := NewConfig(cliCtx)
if err := cfg.Check(); err != nil {
return fmt.Errorf("invalid CLI flags: %w", err)
}
l := oplog.NewLogger(oplog.AppOut(cliCtx), cfg.LogConfig) l := oplog.NewLogger(oplog.AppOut(cliCtx), cfg.LogConfig)
oplog.SetGlobalLogHandler(l.GetHandler()) oplog.SetGlobalLogHandler(l.GetHandler())
......
...@@ -96,6 +96,8 @@ type CLIConfig struct { ...@@ -96,6 +96,8 @@ type CLIConfig struct {
} }
func (c CLIConfig) Check() error { func (c CLIConfig) Check() error {
// TODO: check the sanity of flags loaded directly like MaxPendingTransactions
if err := c.MetricsConfig.Check(); err != nil { if err := c.MetricsConfig.Check(); err != nil {
return err return err
} }
...@@ -105,6 +107,9 @@ func (c CLIConfig) Check() error { ...@@ -105,6 +107,9 @@ func (c CLIConfig) Check() error {
if err := c.TxMgrConfig.Check(); err != nil { if err := c.TxMgrConfig.Check(); err != nil {
return err return err
} }
if err := c.RPCFlag.Check(); err != nil {
return err
}
return nil return nil
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment