Make StatsD configurable

Agent StatsD server is now configurable using the new `enableStatsd` config option. It's disabled by default.
appsignal · Oct 27, 2021 · 1e35cf8 · 1e35cf8
1 parent 67d7fc1
commit 1e35cf8
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 14 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/nodejs/.changesets/add-new-config-option-to-enable-disable-statsd.md b/packages/nodejs/.changesets/add-new-config-option-to-enable-disable-statsd.md
@@ -0,0 +1,5 @@
+---
+bump: "patch"
+---
+
+Add new config option to enable/disable StatsD server in the AppSignal agent. This new config option is called `enableStatsd` and is set to `false` by default. If set to `true`, the AppSignal agent will start a StatsD server on port 8125 on the host.
diff --git a/packages/nodejs/src/config/configmap.ts b/packages/nodejs/src/config/configmap.ts
@@ -23,6 +23,7 @@ export const ENV_TO_KEY_MAPPING: { [key: string]: string } = {
   APPSIGNAL_WORKING_DIRECTORY_PATH: "workingDirectoryPath",
   APPSIGNAL_ENABLE_HOST_METRICS: "enableHostMetrics",
   APPSIGNAL_ENABLE_MINUTELY_PROBES: "enableMinutelyProbes",
+  APPSIGNAL_ENABLE_STATSD: "enableStatsd",
   APPSIGNAL_SKIP_SESSION_DATA: "skipSessionData",
   APPSIGNAL_FILES_WORLD_ACCESSIBLE: "filesWorldAccessible",
   APPSIGNAL_REQUEST_HEADERS: "requestHeaders",
@@ -47,6 +48,7 @@ export const PRIVATE_ENV_MAPPING: { [key: string]: string } = {
   _APPSIGNAL_WORKING_DIR_PATH: "workingDirPath",
   _APPSIGNAL_WORKING_DIRECTORY_PATH: "workingDirectoryPath",
   _APPSIGNAL_ENABLE_HOST_METRICS: "enableHostMetrics",
+  _APPSIGNAL_ENABLE_STATSD: "enableStatsd",
   _APPSIGNAL_HOSTNAME: "hostname",
   _APPSIGNAL_CA_FILE_PATH: "caFilePath",
   _APPSIGNAL_DNS_SERVERS: "dnsServers",
@@ -71,6 +73,7 @@ export const JS_TO_RUBY_MAPPING: { [key: string]: string } = {
   workingDirPath: "working_dir_path",
   workingDirectoryPath: "working_directory_path",
   enableHostMetrics: "enable_host_metrics",
+  enableStatsd: "enable_statsd",
   hostname: "hostname",
   caFilePath: "ca_file_path",
   dnsServers: "dns_servers",

diff --git a/packages/nodejs/src/interfaces/options.ts b/packages/nodejs/src/interfaces/options.ts
@@ -22,6 +22,7 @@ export interface AppsignalOptions {
   workingDirectoryPath: string
   enableHostMetrics: boolean
   enableMinutelyProbes: boolean
+  enableStatsd: boolean
   skipSessionData: boolean
   filesWorldAccessible: boolean
   requestHeaders: string[]

diff --git a/test/integration/diagnose b/test/integration/diagnose