Fixed a bug that prevented storing passwords for new cctray pipelines.

This should fix / help with #13.
ccmenu · Aug 15, 2024 · 14812a5 · 14812a5
1 parent 81df695
commit 14812a5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 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.0</string>
+	<string>25.1</string>
 	<key>CFBundleURLTypes</key>
 	<array>
 		<dict>
@@ -34,7 +34,7 @@
 		</dict>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>2250.1</string>
+	<string>2251.1</string>
 	<key>LSApplicationCategoryType</key>
 	<string>public.app-category.developer-tools</string>
 	<key>LSMinimumSystemVersion</key>

diff --git a/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift b/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift
@@ -46,7 +46,8 @@ class CCTrayPipelineBuilder: ObservableObject {
             components.user = credential.user
             if !credential.password.isEmpty {
                 do {
-                    try Keychain.standard.setPassword(credential.password, forURL: url.absoluteString)
+                    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...
                 }