Merge pull request #2031 from ahoppen/public-logging

Log a few strings publicly

Author: ahoppen

Sources/LanguageServerProtocolExtensions/QueueBasedMessageHandler.swift

@@ -192,7 +192,11 @@ extension QueueBasedMessageHandler {
       )
       .makeSignposter()
       let signpostID = signposter.makeSignpostID()
-      let state = signposter.beginInterval("Notification", id: signpostID, "\(type(of: notification).method)")
+      let state = signposter.beginInterval(
+        "Notification",
+        id: signpostID,
+        "\(type(of: notification).method, privacy: .public)"
+      )
       messageHandlingQueue.async(metadata: DependencyTracker(notification)) {
         signposter.emitEvent("Start handling", id: signpostID)
         await self.handle(notification: notification)
@@ -209,7 +213,7 @@ extension QueueBasedMessageHandler {
     let signposter = Logger(subsystem: LoggingScope.subsystem, category: messageHandlingHelper.signpostLoggingCategory)
       .makeSignposter()
     let signpostID = signposter.makeSignpostID()
-    let state = signposter.beginInterval("Request", id: signpostID, "\(Request.method)")
+    let state = signposter.beginInterval("Request", id: signpostID, "\(Request.method, privacy: .public)")
 
     self.didReceive(request: request, id: id)
 

Sources/SourceKitLSP/SourceKitLSPServer.swift

@@ -751,7 +751,7 @@ extension SourceKitLSPServer: QueueBasedMessageHandler {
       }
     }
 
-    logger.log("Received request \(id): \(params.forLogging)")
+    logger.log("Received request \(id, privacy: .public): \(params.forLogging)")
 
     if let textDocumentRequest = params as? any TextDocumentRequest {
       await self.clientInteractedWithDocument(textDocumentRequest.textDocument.uri)