[CIVIS-2859] Remove application key requirement for ITR

.github/CODEOWNERS

@@ -4,5 +4,5 @@
 #  https://help.github.com/en/articles/about-code-owners
 #
 
-* @nachoBonafonte @DataDog/swift
+* @ypopovych @DataDog/swift
 

IntegrationTests/UITests/IntegrationTestsUITests.xctestplan

@@ -9,7 +9,6 @@
     }
   ],
   "defaultOptions" : {
-    "codeCoverage" : false,
     "environmentVariableEntries" : [
       {
         "key" : "GITHUB_REPOSITORY",

Sources/DatadogSDKTesting/Config.swift

@@ -14,7 +14,6 @@ final class Config {
 
     /// Datatog Configuration values
     var apiKey: String? = nil
-    var applicationKey: String? = nil
     var service: String? = nil
     var tags: [String: String] = [:]
     var customConfigurations: [String: String] = [:]
@@ -77,7 +76,6 @@ final class Config {
         isEnabled = env.has(.isEnabled) ? env[.isEnabled] ?? false : nil
 
         apiKey = env[.apiKey] ?? env.get(info: "DatadogApiKey")
-        applicationKey = env[.applicationKey] ?? env[.appKey] ?? env.get(info: "DatadogApplicationKey")
         environment = env[.environment]
 
         let tracerUnderTesting = env.has(env: .testOutputFile)

Sources/DatadogSDKTesting/DDTestMonitor.swift

@@ -281,12 +281,6 @@ internal class DDTestMonitor {
                     // Activate Intelligent Test Runner
                     if itrBackendConfig?.testsSkipping ?? false {
                         Log.debug("ITR Enabled")
-
-                        if DDTestMonitor.config.applicationKey == nil {
-                            Log.print("APPLICATION_KEY env variable is not set, this is needed for Intelligent Test Runner")
-                            itr = nil
-                            return
-                        }
 
                         itr = IntelligentTestRunner(configurations: DDTestMonitor.baseConfigurationTags)
                         itr?.start()

Sources/DatadogSDKTesting/DDTracer.swift

@@ -96,7 +96,6 @@ internal class DDTracer {
             environment: env.environment,
             hostname: hostnameToReport,
             apiKey: conf.apiKey ?? "",
-            applicationKey: conf.applicationKey ?? "",
             endpoint: endpoint,
             payloadCompression: payloadCompression,
             performancePreset: .instantDataDelivery,

Sources/DatadogSDKTesting/Environment/EnvironmentKeys.swift

@@ -41,8 +41,6 @@ internal enum EnvironmentKey: String, CaseIterable {
     case ciVisibilityExcludedBranches = "DD_CIVISIBILITY_EXCLUDED_BRANCHES"
     case ciVisibilityReportHostname = "DD_CIVISIBILITY_REPORT_HOSTNAME"
     case disableSourceLocation = "DD_DISABLE_SOURCE_LOCATION"
-    case applicationKey = "DD_APPLICATION_KEY"
-    case appKey = "DD_APP_KEY"
     case localTestEnvironmentPort = "DD_LOCAL_TEST_ENVIRONMENT_PORT"
     case disableGitInformation = "DD_DISABLE_GIT_INFORMATION"
     case testOutputFile = "TEST_OUTPUT_FILE"

Sources/DatadogSDKTesting/Utils/Clock.swift

@@ -77,7 +77,7 @@ final class NTPClock: Clock {
                     self._state = .synced({ Kronos.Clock.now! })
                 } else {
                     self._state = .synced({ Date() })
-                    Log.debug("NTP server sync failed")
+                    Log.print("NTP server sync failed")
                 }
                 waiter.signal()
             }

Sources/EventsExporter/CoverageExporter/CoverageExporter.swift

@@ -62,7 +62,7 @@ internal class CoverageExporter {
         do {
             profData = try DDCoverageConversor.generateProfData(profrawFile: coverage)
         } catch {
-            Log.debug("Prof Data genetation failed: \(error)")
+            Log.print("Profiler Data genetation failed: \(error)")
             return
         }
 

Sources/EventsExporter/ExporterConfiguration.swift

@@ -21,7 +21,6 @@ public struct ExporterConfiguration {
 
     /// API key for authentication
     var apiKey: String
-    var applicationKey: String
 
     /// Endpoint that will be used for reporting.
     var endpoint: Endpoint
@@ -37,15 +36,14 @@ public struct ExporterConfiguration {
 
     var logger: Logger
 
-    public init(serviceName: String, libraryVersion: String, applicationName: String, applicationVersion: String, environment: String, hostname: String?, apiKey: String, applicationKey: String, endpoint: Endpoint, payloadCompression: Bool = true, source: String = "ios", performancePreset: PerformancePreset = .default, exporterId: String, logger: Logger) {
+    public init(serviceName: String, libraryVersion: String, applicationName: String, applicationVersion: String, environment: String, hostname: String?, apiKey: String, endpoint: Endpoint, payloadCompression: Bool = true, source: String = "ios", performancePreset: PerformancePreset = .default, exporterId: String, logger: Logger) {
         self.serviceName = serviceName
         self.libraryVersion = libraryVersion
         self.applicationName = applicationName
         self.version = applicationVersion
         self.environment = environment
         self.hostname = hostname
         self.apiKey = apiKey
-        self.applicationKey = applicationKey
         self.endpoint = endpoint
         self.payloadCompression = payloadCompression
         self.source = source

Sources/EventsExporter/IntelligentTestRunner/ITRService.swift

@@ -29,7 +29,6 @@ internal class ITRService {
                     device: Device.current
                 ),
                 .apiKeyHeader(apiKey: config.apiKey),
-                .applicationKeyHeader(applicationKey: config.applicationKey),
                 .traceIDHeader(traceID: config.exporterId),
                 .parentSpanIDHeader(parentSpanID: config.exporterId),
                 .samplingPriorityHeader()
@@ -46,7 +45,6 @@ internal class ITRService {
                     device: Device.current
                 ),
                 .apiKeyHeader(apiKey: config.apiKey),
-                .applicationKeyHeader(applicationKey: config.applicationKey),
                 .traceIDHeader(traceID: config.exporterId),
                 .parentSpanIDHeader(parentSpanID: config.exporterId),
                 .samplingPriorityHeader()
@@ -63,7 +61,6 @@ internal class ITRService {
                     device: Device.current
                 ),
                 .apiKeyHeader(apiKey: config.apiKey),
-                .applicationKeyHeader(applicationKey: config.applicationKey),
                 .traceIDHeader(traceID: config.exporterId),
                 .parentSpanIDHeader(parentSpanID: config.exporterId),
                 .samplingPriorityHeader()
@@ -80,7 +77,6 @@ internal class ITRService {
                     device: Device.current
                 ),
                 .apiKeyHeader(apiKey: config.apiKey),
-                .applicationKeyHeader(applicationKey: config.applicationKey),
                 .traceIDHeader(traceID: config.exporterId),
                 .parentSpanIDHeader(parentSpanID: config.exporterId),
                 .samplingPriorityHeader()

Sources/EventsExporter/Upload/HTTPHeader.swift

@@ -65,11 +65,6 @@ struct HTTPHeader {
         return HTTPHeader(field: aPIKeyHeaderField, value: .constant(apiKey))
     }
 
-    /// Datadog request Application Key authentication header.
-    static func applicationKeyHeader(applicationKey: String) -> HTTPHeader {
-        return HTTPHeader(field: applicationKeyHeaderField, value: .constant(applicationKey))
-    }
-
     /// Datadog request Application Key authentication header.
     static func hostnameHeader(hostname: String) -> HTTPHeader {
         return HTTPHeader(field: hostnameHeaderField, value: .constant(hostname))

Tests/EventsExporter/EventsExporterTests.swift

@@ -53,7 +53,6 @@ class EventsExporterTests: XCTestCase {
                                                           environment: "environment",
                                                           hostname: "hostname",
                                                           apiKey: "apikey",
-                                                          applicationKey: "applicationkey",
                                                           endpoint: Endpoint.custom(
                                                               testsURL: URL(string: "http://localhost:33333/traces")!,
                                                               logsURL: URL(string: "http://localhost:33333/logs")!

Tests/EventsExporter/Logs/LogsExporterTests.swift

@@ -43,7 +43,6 @@ class LogsExporterTests: XCTestCase {
                                                   environment: "environment",
                                                   hostname: "hostname",
                                                   apiKey: "apikey",
-                                                  applicationKey: "applicationkey",
                                                   endpoint: Endpoint.custom(
                                                       testsURL: URL(string: "http://localhost:33333/traces")!,
                                                       logsURL: URL(string: "http://localhost:33333/logs")!

Tests/EventsExporter/Spans/SpansExporterTests.swift

@@ -44,7 +44,6 @@ class SpansExporterTests: XCTestCase {
                                                   environment: "environment",
                                                   hostname: nil,
                                                   apiKey: "apikey",
-                                                  applicationKey: "applicationkey",
                                                   endpoint: Endpoint.custom(
                                                       testsURL: URL(string: "http://localhost:33333/traces")!,
                                                       logsURL: URL(string: "http://localhost:33333/logs")!