Use feedback handler instead of print() (#1040)

Author: edigaryev

Sources/Exporters/OpenTelemetryProtocolGrpc/trace/OtlpTraceJsonExporter.swift

@@ -4,6 +4,7 @@
  */
 
 import Foundation
+import OpenTelemetryApi
 import OpenTelemetrySdk
 import OpenTelemetryProtocolExporterCommon
 
@@ -32,7 +33,7 @@ public class OtlpTraceJsonExporter: SpanExporter {
         let span = try JSONDecoder().decode(OtlpSpan.self, from: jsonData)
         exportedSpans.append(span)
       } catch {
-        print("Decode Error: \(error)")
+        OpenTelemetry.instance.feedbackHandler?("Decode Error: \(error)")
       }
       return .success
     } catch {

Sources/Exporters/Prometheus/PrometheusExporterHttpServer.swift

@@ -6,6 +6,7 @@
 import Foundation
 import NIO
 import NIOHTTP1
+import OpenTelemetryApi
 
 public class PrometheusExporterHttpServer {
   private let group = MultiThreadedEventLoopGroup(numberOfThreads: 1)
@@ -24,7 +25,7 @@ public class PrometheusExporterHttpServer {
   public func start() throws {
     do {
       let channel = try serverBootstrap.bind(host: host, port: port).wait()
-      print("Listening on \(String(describing: channel.localAddress))...")
+      OpenTelemetry.instance.feedbackHandler?("Listening on \(String(describing: channel.localAddress))...")
       try channel.closeFuture.wait()
     } catch {
       throw error
@@ -35,10 +36,10 @@ public class PrometheusExporterHttpServer {
     do {
       try group.syncShutdownGracefully()
     } catch {
-      print("Error shutting down \(error.localizedDescription)")
+      OpenTelemetry.instance.feedbackHandler?("Error shutting down \(error.localizedDescription)")
       exit(0)
     }
-    print("Client connection closed")
+    OpenTelemetry.instance.feedbackHandler?("Client connection closed")
   }
 
   private var serverBootstrap: ServerBootstrap {
@@ -108,7 +109,7 @@ public class PrometheusExporterHttpServer {
     }
 
     public func errorCaught(context: ChannelHandlerContext, error: Error) {
-      print("error: ", error)
+      OpenTelemetry.instance.feedbackHandler?("error: \(error)")
 
       // As we are not really interested getting notified on success or failure we just pass nil as promise to
       // reduce allocations.

Sources/Instrumentation/SDKResourceExtension/DataSource/DeviceDataSource.swift

@@ -4,6 +4,7 @@
  */
 
 import Foundation
+import OpenTelemetryApi
 #if os(watchOS)
   import WatchKit
 #elseif os(macOS)
@@ -47,7 +48,7 @@ public class DeviceDataSource: IDeviceDataSource {
       let modelRequestError = sysctl(hwName, 2, machine, len, nil, 0)
       if modelRequestError != 0 {
         // TODO: better error log
-        print("error #\(errno): \(String(describing: String(utf8String: strerror(errno))))")
+        OpenTelemetry.instance.feedbackHandler?("error #\(errno): \(String(describing: String(utf8String: strerror(errno))))")
 
         return nil
       }

Sources/Instrumentation/URLSession/URLSessionInstrumentation.swift

@@ -170,7 +170,7 @@ public class URLSessionInstrumentation {
     ].forEach {
       let selector = $0
       guard let original = class_getInstanceMethod(cls, selector) else {
-        print("injectInto \(selector.description) failed")
+        OpenTelemetry.instance.feedbackHandler?("injectInto \(selector.description) failed")
         return
       }
       var originalIMP: IMP?
@@ -295,7 +295,7 @@ public class URLSessionInstrumentation {
     ].forEach {
       let selector = $0
       guard let original = class_getInstanceMethod(cls, selector) else {
-        print("injectInto \(selector.description) failed")
+        OpenTelemetry.instance.feedbackHandler?("injectInto \(selector.description) failed")
         return
       }
       var originalIMP: IMP?
@@ -385,7 +385,7 @@ public class URLSessionInstrumentation {
     ].forEach {
       let selector = $0
       guard let original = class_getInstanceMethod(cls, selector) else {
-        print("injectInto \(selector.description) failed")
+        OpenTelemetry.instance.feedbackHandler?("injectInto \(selector.description) failed")
         return
       }
       var originalIMP: IMP?