Change dsn to nullable

Mikroservices · Feb 16, 2024 · 18c5dcc · 18c5dcc
1 parent 74b1c44
commit 18c5dcc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 11 deletions.
diff --git a/Sources/ExtendedLogging/SentryLogger.swift b/Sources/ExtendedLogging/SentryLogger.swift
@@ -14,7 +14,7 @@ public struct SentryLogger: LogHandler {
     private var logFormatter: LogFormatter
 
     public init(label: String,
-                dsn: String,
+                dsn: String?,
                 level: Logger.Level = .debug,
                 metadata: Logger.Metadata = [:]) {
         self.label = label
@@ -52,9 +52,4 @@ public struct SentryLogger: LogHandler {
             self.metadata[metadataKey] = newValue
         }
     }
-
-    // for testing
-    public func wait() -> Void {
-        // fileWriter.wait()
-    }
 }
diff --git a/Sources/ExtendedLogging/SentryWriter.swift b/Sources/ExtendedLogging/SentryWriter.swift
@@ -1,20 +1,24 @@
 import Foundation
 
 internal class SentryWriter {
-    private let dsn: String;
+    private let dsn: String?
     private let requestQueue = DispatchQueue.init(label: "SentryWriter", qos: .utility)
 
-    init(dsn: String) {
+    init(dsn: String?) {
         self.dsn = dsn
     }
 
     func write(message: Data?) {
+        guard let sentryDsn = self.dsn else {
+            return
+        }
+
         guard let data = message else {
             print("[SentryWriter] Logged message cannot be nil.")
             return
         }
 
-        guard let url = self.convertToUrl() else {
+        guard let url = self.convertToUrl(sentryDsn: sentryDsn) else {
             print("[SentryWriter] Cannot parse Sentry DSN url.")
             return
         }
@@ -33,8 +37,8 @@ internal class SentryWriter {
         }
     }
 
-    private func convertToUrl() -> URL? {
-        let dsnUrl = URLComponents(string: self.dsn)
+    private func convertToUrl(sentryDsn: String) -> URL? {
+        let dsnUrl = URLComponents(string: sentryDsn)
 
         guard let scheme = dsnUrl?.scheme else {
             return nil