feat: Add Logging Configurations (#1058)

Author: metal-messiah

src/common/config/state/init.js

@@ -31,12 +31,6 @@ const model = () => {
     }
   }
   return {
-    feature_flags: [],
-    proxy: {
-      assets: undefined, // if this value is set, it will be used to overwrite the webpack asset path used to fetch assets
-      beacon: undefined // likewise for the url to which we send analytics
-    },
-    privacy: { cookies_enabled: true }, // *cli - per discussion, default should be true
     ajax: { deny_list: undefined, block_internal: true, enabled: true, harvestTimeSeconds: 10, autoStart: true },
     distributed_tracing: {
       enabled: undefined,
@@ -45,19 +39,24 @@ const model = () => {
       cors_use_tracecontext_headers: undefined,
       allowed_origins: undefined
     },
-    session: {
-      expiresMs: DEFAULT_EXPIRES_MS,
-      inactiveMs: DEFAULT_INACTIVE_MS
-    },
-    ssl: undefined,
-    obfuscate: undefined,
+    feature_flags: [],
+    harvest: { tooManyRequestsDelay: 60 },
     jserrors: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
+    logging: { enabled: true, harvestTimeSeconds: 30, autoStart: true, level: 'info' },
     metrics: { enabled: true, autoStart: true },
+    obfuscate: undefined,
     page_action: { enabled: true, harvestTimeSeconds: 30, autoStart: true },
     page_view_event: { enabled: true, autoStart: true },
     page_view_timing: { enabled: true, harvestTimeSeconds: 30, long_task: false, autoStart: true },
-    session_trace: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
-    harvest: { tooManyRequestsDelay: 60 },
+    privacy: { cookies_enabled: true }, // *cli - per discussion, default should be true
+    proxy: {
+      assets: undefined, // if this value is set, it will be used to overwrite the webpack asset path used to fetch assets
+      beacon: undefined // likewise for the url to which we send analytics
+    },
+    session: {
+      expiresMs: DEFAULT_EXPIRES_MS,
+      inactiveMs: DEFAULT_INACTIVE_MS
+    },
     session_replay: {
       // feature settings
       autoStart: true,
@@ -100,8 +99,10 @@ const model = () => {
         else warn('An invalid session_replay.mask_input_option was provided and will not be used', val)
       }
     },
+    session_trace: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
+    soft_navigations: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
     spa: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
-    soft_navigations: { enabled: true, harvestTimeSeconds: 10, autoStart: true }
+    ssl: undefined
   }
 }
 

tests/unit/common/config/state/init.test.js

@@ -30,6 +30,123 @@ test('getConfigurationValue parses path correctly', () => {
   expect(getConfigurationValue('ab', 'page_action.dne')).toBeUndefined()
 })
 
+test('init props exist and return expected defaults', () => {
+  setConfiguration('34567', {})
+  const config = getConfiguration('34567')
+  expect(Object.keys(config).length).toEqual(19)
+  expect(config.ajax).toEqual({
+    autoStart: true,
+    block_internal: true,
+    deny_list: undefined,
+    enabled: true,
+    harvestTimeSeconds: 10
+  })
+  expect(config.distributed_tracing).toEqual({
+    allowed_origins: undefined,
+    cors_use_newrelic_header: undefined,
+    cors_use_tracecontext_headers: undefined,
+    enabled: undefined,
+    exclude_newrelic_header: undefined
+  })
+  expect(config.feature_flags).toEqual([])
+  expect(config.harvest).toEqual({
+    tooManyRequestsDelay: 60
+  })
+  expect(config.jserrors).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 10
+  })
+  expect(config.logging).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 30,
+    level: 'info'
+  })
+  expect(config.metrics).toEqual({
+    autoStart: true,
+    enabled: true
+  })
+  expect(config.obfuscate).toEqual(undefined)
+  expect(config.page_action).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 30
+  })
+  expect(config.page_view_event).toEqual({
+    autoStart: true,
+    enabled: true
+  })
+  expect(config.page_view_timing).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 30,
+    long_task: false
+  })
+  expect(config.privacy).toEqual({
+    cookies_enabled: true
+  })
+  expect(config.proxy).toEqual({
+    assets: undefined,
+    beacon: undefined
+  })
+  expect(config.session).toEqual({
+    expiresMs: 14400000,
+    inactiveMs: 1800000
+  })
+  expect(config.session_replay).toEqual({
+    autoStart: true,
+    block_class: 'nr-block',
+    block_selector: '[data-nr-block]',
+    collect_fonts: false,
+    enabled: false,
+    error_sampling_rate: 100,
+    harvestTimeSeconds: 60,
+    ignore_class: 'nr-ignore',
+    inline_images: false,
+    inline_stylesheet: true,
+    mask_all_inputs: true,
+    mask_input_options: {
+      color: false,
+      date: false,
+      'datetime-local': false,
+      email: false,
+      month: false,
+      number: false,
+      password: true,
+      range: false,
+      search: false,
+      select: false,
+      tel: false,
+      text: false,
+      textarea: false,
+      time: false,
+      url: false,
+      week: false
+    },
+    mask_text_class: 'nr-mask',
+    mask_text_selector: '*',
+    preload: false,
+    sampling_rate: 10
+  })
+  expect(config.session_trace).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 10
+  })
+  expect(config.soft_navigations).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 10
+  })
+  expect(config.spa).toEqual({
+    autoStart: true,
+    enabled: true,
+    harvestTimeSeconds: 10
+  })
+  expect(config.ssl).toEqual(undefined)
+})
+
 describe('property getters/setters used for validation', () => {
   test('invalid values do not pass through', () => {
     setConfiguration('12345', {