Fix typo and remove not needed headers

Author: iunanua

packages/dd-trace/src/appsec/iast/telemetry/logs.js

@@ -28,7 +28,7 @@ function sendLogs () {
 }
 
 function isLevelEnabled (level) {
-  return isLogCollectionEnabled(config) && (level !== 'DEBUG' || config.telemetry.diagnosticLogCollector)
+  return isLogCollectionEnabled(config) && (level !== 'DEBUG' || config.telemetry.diagnosticLogCollection)
 }
 
 function isLogCollectionEnabled (config) {

packages/dd-trace/src/telemetry/send-data.js

@@ -1,6 +1,8 @@
 const request = require('../exporters/common/request')
 
-function getHeaders (reqType, debug, application) {
+function getHeaders (reqType, config) {
+  const debug = config.telemetry && config.telemetry.debug
+
   const headers = {
     'content-type': 'application/json',
     'dd-telemetry-api-version': 'v1',
@@ -9,10 +11,6 @@ function getHeaders (reqType, debug, application) {
   if (debug) {
     headers['dd-telemetry-debug-enabled'] = 'true'
   }
-  if (application) {
-    headers['dd-client-library-language'] = application.language_name
-    headers['dd-client-library-version'] = application.tracer_version
-  }
   return headers
 }
 
@@ -36,14 +34,13 @@ function sendData (config, application, host, reqType, payload = {}) {
     url
   } = config
 
-  const debug = config.telemetry && config.telemetry.debug
   const options = {
     url,
     hostname,
     port,
     method: 'POST',
     path: '/telemetry/proxy/api/v2/apmtelemetry',
-    headers: getHeaders(reqType, debug, application)
+    headers: getHeaders(reqType, config)
   }
   const data = JSON.stringify({
     api_version: 'v1',

packages/dd-trace/test/appsec/iast/iast-log.spec.js

@@ -10,7 +10,7 @@ describe('IAST log', () => {
     config: {
       telemetry: {
         logCollection: true,
-        diagnosticLogCollector: true
+        diagnosticLogCollection: true
       }
     }
   }
@@ -19,7 +19,7 @@ describe('IAST log', () => {
     config: {
       telemetry: {
         logCollection: true,
-        diagnosticLogCollector: false
+        diagnosticLogCollection: false
       }
     }
   }

packages/dd-trace/test/appsec/iast/telemetry/logs.spec.js

@@ -134,7 +134,7 @@ describe('telemetry logs', () => {
     const app = {}
     const host = {}
 
-    it('should be called with DEBUG level and error if config.telemetry.diagnosticLogCollector = true', () => {
+    it('should be called with DEBUG level and error if config.telemetry.diagnosticLogCollection = true', () => {
       const logCollectorAdd = sinon.stub()
       const logs = proxyquire('../../../../src/appsec/iast/telemetry/logs', {
         '../../../../../diagnostics_channel': dc,
@@ -144,7 +144,7 @@ describe('telemetry logs', () => {
       })
       logs.start()
 
-      onTelemetryStartMsg.config.telemetry.diagnosticLogCollector = true
+      onTelemetryStartMsg.config.telemetry.diagnosticLogCollection = true
       onTelemetryStartMsg.application = app
       onTelemetryStartMsg.host = host
       onTelemetryStart()(onTelemetryStartMsg)
@@ -156,7 +156,7 @@ describe('telemetry logs', () => {
       expect(logCollectorAdd).to.be.calledOnceWith(match({ message: 'test', level: 'DEBUG', stack_trace: stack }))
     })
 
-    it('should be not called with DEBUG level if config.telemetry.diagnosticLogCollector = false', () => {
+    it('should be not called with DEBUG level if config.telemetry.diagnosticLogCollection = false', () => {
       const logCollectorAdd = sinon.stub()
       const logs = proxyquire('../../../../src/appsec/iast/telemetry/logs', {
         '../../../../../diagnostics_channel': dc,

packages/dd-trace/test/telemetry/send-data.spec.js

@@ -34,9 +34,7 @@ describe('sendData', () => {
       headers: {
         'content-type': 'application/json',
         'dd-telemetry-api-version': 'v1',
-        'dd-telemetry-request-type': 'req-type',
-        'dd-client-library-language': application.language_name,
-        'dd-client-library-version': application.tracer_version
+        'dd-telemetry-request-type': 'req-type'
       },
       url: undefined,
       hostname: '',
@@ -59,9 +57,7 @@ describe('sendData', () => {
       headers: {
         'content-type': 'application/json',
         'dd-telemetry-api-version': 'v1',
-        'dd-telemetry-request-type': 'req-type',
-        'dd-client-library-language': application.language_name,
-        'dd-client-library-version': application.tracer_version
+        'dd-telemetry-request-type': 'req-type'
       },
       url: 'unix:/foo/bar/baz',
       hostname: undefined,
@@ -86,9 +82,7 @@ describe('sendData', () => {
         'content-type': 'application/json',
         'dd-telemetry-api-version': 'v1',
         'dd-telemetry-request-type': 'req-type',
-        'dd-telemetry-debug-enabled': 'true',
-        'dd-client-library-language': application.language_name,
-        'dd-client-library-version': application.tracer_version
+        'dd-telemetry-debug-enabled': 'true'
       },
       url: '/test',
       hostname: undefined,