Added Config singleton class to unify configuration parameter access and instantiation

Author: holgerd77

bin/cli.ts

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 
-const Common = require('@ethereumjs/common').default
 const chains = require('@ethereumjs/common/dist/chains').chains
 const { getLogger } = require('../lib/logging')
 const { parseParams, parseTransports } = require('../lib/util')
 const { fromName: serverFromName } = require('../lib/net/server')
 import Node from '../lib/node'
 import { Server as RPCServer } from 'jayson'
+import { Config } from '../lib/config'
 const RPCManager = require('../lib/rpc')
 const level = require('level')
 const os = require('os')
@@ -95,7 +95,7 @@ async function runNode(options: any) {
   node.on('synchronized', () => {
     logger.info('Synchronized')
   })
-  logger.info(`Connecting to network: ${options.common.chainName()}`)
+  logger.info(`Connecting to network: ${options.config.common.chainName()}`)
   await node.open()
   logger.info('Synchronizing blockchain...')
   await node.start()
@@ -128,16 +128,11 @@ async function run() {
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
   const chainParams = args.params ? await parseParams(args.params) : args.network
 
-  // Initialize Common with an explicit 'chainstart' HF set until
-  // hardfork awareness is implemented within the library
-  // Also a fix for https://github.com/ethereumjs/ethereumjs-vm/issues/757
-
-  // TODO: map chainParams (and lib/util.parseParams) to new Common format
-  const common = new Common({ chain: args.network, hardfork: 'chainstart' })
+  const config = new Config()
   const servers = parseTransports(args.transports).map((t: any) => {
     const Server = serverFromName(t.name)
     if (t.name === 'rlpx') {
-      t.options.bootnodes = t.options.bootnodes || common.bootstrapNodes()
+      t.options.bootnodes = t.options.bootnodes || config.common.bootstrapNodes()
     }
     return new Server({ logger, ...t.options })
   })
@@ -147,7 +142,7 @@ async function run() {
   logger.info(`Data directory: ${dataDir}`)
 
   const options = {
-    common,
+    config,
     logger,
     servers,
     syncmode: args.syncmode,

lib/config.ts

@@ -0,0 +1,35 @@
+import Common from '@ethereumjs/common'
+
+export interface Options {
+  /**
+   * Specify the chain and hardfork by passing a Common instance.
+   *
+   * If not provided this defaults to chain `mainnet` and hardfork `chainstart`
+   */
+  common?: Common
+} 
+
+export class Config {
+  
+  public common!: Common
+
+  static instance: Config
+
+  constructor(options: Options = {}) {
+    if (Config.instance) {
+      return Config.instance
+    }
+
+    // Initialize Common with an explicit 'chainstart' HF set until
+    // hardfork awareness is implemented within the library
+    // Also a fix for https://github.com/ethereumjs/ethereumjs-vm/issues/757
+
+    // TODO: map chainParams (and lib/util.parseParams) to new Common format
+    this.common = options.common || new Common({ chain: 'mainnet', hardfork: 'chainstart' })
+
+    Config.instance = this
+  }
+
+
+
+}

lib/node.ts

@@ -1,6 +1,7 @@
 import * as events from 'events'
 import { FastEthereumService, LightEthereumService } from './service'
 import { defaultLogger } from './logging'
+import { Config } from './config'
 
 const defaultOptions = {
   minPeers: 3,
@@ -15,8 +16,9 @@ const defaultOptions = {
  * @memberof module:node
  */
 export default class Node extends events.EventEmitter {
+  public config: Config
+  
   public logger: any
-  public common: any
   public servers: any
   public syncmode: any
   public services: any
@@ -27,8 +29,8 @@ export default class Node extends events.EventEmitter {
   /**
    * Create new node
    * @param {Object}   options constructor parameters
+   * @param {Config}   [options.config] Client configuration
    * @param {Logger}   [options.logger] Logger instance
-   * @param {Common}   [options.common] common parameters
    * @param {LevelDB}  [options.db=null] blockchain database
    * @param {string}   [options.syncmode=light] synchronization mode ('fast' or 'light')
    * @param {boolean}  [options.lightserv=false] serve LES requests
@@ -42,9 +44,8 @@ export default class Node extends events.EventEmitter {
   constructor(options: any) {
     super()
     options = { ...defaultOptions, ...options }
-
+    this.config = options.config || new Config()
     this.logger = options.logger
-    this.common = options.common
     this.servers = options.servers
     this.syncmode = options.syncmode
     this.services = [
@@ -53,15 +54,15 @@ export default class Node extends events.EventEmitter {
             servers: this.servers,
             logger: this.logger,
             lightserv: options.lightserv,
-            common: options.common,
+            common: this.config.common,
             minPeers: options.minPeers,
             maxPeers: options.maxPeers,
             db: options.db,
           })
         : new LightEthereumService({
             servers: this.servers,
             logger: this.logger,
-            common: options.common,
+            common: this.config.common,
             minPeers: options.minPeers,
             maxPeers: options.maxPeers,
             db: options.db,

lib/types.ts

@@ -0,0 +1,20 @@
+import type { LevelUp } from 'levelup'
+import { Logger } from 'winston'
+import Common from '@ethereumjs/common'
+
+export interface Options {
+    /**
+     * Specify the chain and hardfork by passing a Common instance.
+     *
+     * If not provided this defaults to chain `mainnet` and hardfork `chainstart`
+     */
+    common?: Common,
+    /**
+     * Database to store blocks and metadata. Should be an abstract-leveldown compliant store.
+     */
+    db?: LevelUp
+    /**
+     * Logger (winston) with log level used
+     */
+    logger?: Logger,
+}