Skip to content

Commit 5b4880e

Browse files
committed
Tweeks to get logging to work
Signed-off-by: Kevin Earls <[email protected]>
1 parent 5ce07e1 commit 5b4880e

File tree

3 files changed

+5
-3
lines changed

3 files changed

+5
-3
lines changed

Sources/Exporters/OpenTelemetryProtocol/logs/OtlpHttpLogExporter.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,14 @@ public class OtlpHttpLogExporter : LogRecordExporter {
3737
}
3838

3939
public func flush() -> ExportResult {
40-
var exporterResult: ExportResult = .success
40+
var exporterResult: ExportResult = .failure
4141
let logRecords = pendingLogRecords
4242
pendingLogRecords = []
4343

4444
let body = Opentelemetry_Proto_Collector_Logs_V1_ExportLogsServiceRequest.with { request in
4545
request.resourceLogs = LogRecordAdapter.toProtoResourceRecordLog(logRecordList: logRecords)
4646
}
47-
47+
4848
var request = URLRequest(url: endpoint)
4949
request.httpMethod = "POST"
5050
request.httpBody = try? body.serializedData()

Sources/Exporters/OpenTelemetryProtocol/trace/OtlpHTttpTraceExporter.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ public class OtlpHttpTraceExporter: SpanExporter {
2626
}
2727

2828
public func flush() -> SpanExporterResultCode {
29-
var exporterResult: SpanExporterResultCode = .success
29+
var exporterResult: SpanExporterResultCode = .failure
3030
let spans = pendingSpans
3131
pendingSpans = []
3232
let body = Opentelemetry_Proto_Collector_Trace_V1_ExportTraceServiceRequest.with {

Sources/Exporters/OpenTelemetryProtocol/trace/utils/HTTPClient.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ internal final class HTTPClient {
2828
let task = session.dataTask(with: request) { data, response, error in
2929
completion(httpClientResult(for: (data, response, error)))
3030
}
31+
3132
task.resume()
33+
Thread.sleep(forTimeInterval: 1) // FIXME is there a better way to handle this?
3234
}
3335
}
3436

0 commit comments

Comments
 (0)