[#371] Add ConfigurationBuilder for flexible config management

Author: feelform

lib/config.d.ts

@@ -46,4 +46,3 @@ export declare function readConfigJson(formattedConfig: any): Partial<PinpointCo
 export declare function readRootConfigFile(): { [key: string]: any };
 export declare function getMainModulePath(requireFunction: NodeRequire): string | undefined;
 export declare function isContainerEnvironment(): boolean;
-export declare function initializeConfig(initOptions?: { [key: string]: any }): void;

lib/config.js

@@ -283,20 +283,6 @@ const getValue = (key, configFile) => {
   }
 }
 
-const getConfig = (initOptions) => {
-  if (!agentConfig) {
-    init(initOptions)
-  }
-  return agentConfig
-}
-
-const initializeConfig = (initOptions) => {
-  clear()
-  init(initOptions)
-}
-
-const clear = () => agentConfig && (agentConfig = null)
-
 //https://github.com/sindresorhus/is-docker
 const isContainerEnvironment = () => {
   return hasDockerEnv() || hasDockerCGroup() || (process.env['KUBERNETES_SERVICE_HOST'] && process.env['KUBERNETES_SERVICE_HOST'].length > 0)
@@ -462,13 +448,10 @@ const setConfig = (config) => {
 const clear2 = () => configInstance && (configInstance = null)
 
 module.exports = {
-  getConfig,
-  clear,
   readConfigJson,
   readRootConfigFile,
   getMainModulePath,
   isContainerEnvironment,
-  initializeConfig,
   ConfigBuilder,
   getConfig2,
   setConfig,

test/fixture.js

@@ -8,7 +8,7 @@ const TransactionId = require('../lib/context/transaction-id')
 const TraceId = require('../lib/context/trace-id')
 const SpanId = require('../lib/context/span-id')
 const testConfig= require('./pinpoint-config-test')
-require('../lib/config').clear()
+require('../lib/config').clear2()
 const config = require('../lib/config').getConfig2(testConfig)
 const { namedGroupLocationFileName, namedGroupTypeMethod } = require('../lib/instrumentation/call-stack')