Added logging to help diagnose issue #13.

ccmenu · Aug 16, 2024 · f326d27 · f326d27
1 parent 69c8259
commit f326d27
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 6 deletions.
diff --git a/CCMenu/Resources/Info.plist b/CCMenu/Resources/Info.plist
@@ -19,7 +19,7 @@
 	<key>CFBundlePackageType</key>
 	<string>$(PRODUCT_BUNDLE_PACKAGE_TYPE)</string>
 	<key>CFBundleShortVersionString</key>
-	<string>25.1</string>
+	<string>25.2d1</string>
 	<key>CFBundleURLTypes</key>
 	<array>
 		<dict>
@@ -34,7 +34,7 @@
 		</dict>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>2251.3</string>
+	<string>2252.1</string>
 	<key>LSApplicationCategoryType</key>
 	<string>public.app-category.developer-tools</string>
 	<key>LSMinimumSystemVersion</key>

diff --git a/CCMenu/Source/Miscellaneous/Keychain.swift b/CCMenu/Source/Miscellaneous/Keychain.swift
@@ -6,7 +6,7 @@
 
 import Foundation
 import Security
-
+import os
 
 enum KeychainAccessError: Error {
     case passwordEncodingErr
@@ -50,9 +50,12 @@ class Keychain {
     }
 
     func getPassword(forURL url: URL) throws -> String? {
+        let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "keychain")
         if let password = cache[url.absoluteString] {
+            logger.log("Using cached password for \(url)")
             return password
         }
+        logger.log("Retrieving password for \(url) from keychain")
         let query = [
             kSecClass:        kSecClassInternetPassword,
             kSecAttrServer:   try getOrThrow(error: .missingHostErr) { url.host() },
@@ -62,6 +65,11 @@ class Keychain {
             kSecReturnData:   true
         ] as NSDictionary
         let password = try getStringForQuery(query)
+        if let password {
+            logger.log("Got password (length = \(password.count))")
+        } else {
+            logger.log("Didn't get a password")
+        }
         cache[url.absoluteString] = password
         return password
     }

diff --git a/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift b/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift
@@ -4,8 +4,8 @@
  *  not use these files except in compliance with the License.
  */
 
-
 import Foundation
+import os
 
 class CCTrayPipelineBuilder: ObservableObject {
 
@@ -49,7 +49,8 @@ class CCTrayPipelineBuilder: ObservableObject {
                     let newUrl = components.url?.absoluteURL ?? url
                     try Keychain.standard.setPassword(credential.password, forURL: newUrl.absoluteString)
                 } catch {
-                    // TODO: Figure out what to do here – so many errors...
+                    let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "keychain")
+                    logger.error("Error when storing password in keychain: \(error.localizedDescription)")
                 }
             }
         } else {

diff --git a/CCMenu/Source/Server Monitor/CCTrayAPI.swift b/CCMenu/Source/Server Monitor/CCTrayAPI.swift
@@ -5,7 +5,7 @@
  */
 
 import Foundation
-
+import os
 
 struct HTTPCredential {
     var user: String
@@ -19,11 +19,16 @@ struct HTTPCredential {
 class CCTrayAPI {
 
     static func requestForProjects(url: URL, credential: HTTPCredential?) -> URLRequest {
+        let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "network")
         var request = URLRequest(url: url)
 
         if let credential {
             let v = URLRequest.basicAuthValue(user: credential.user, password: credential.password)
             request.setValue(v, forHTTPHeaderField: "Authorization")
+            let redacted = v.replacingOccurrences(of: "[A-Za-z0-9=]", with: "*", options: [.regularExpression])
+            logger.log("Making request for url \(url) with authorization \(redacted)")
+        } else {
+            logger.log("Making request for url \(url)")
         }
 
         return request