From 14812a5484c3cf73897d86a5463abeab31a6a4f0 Mon Sep 17 00:00:00 2001 From: Erik Doernenburg Date: Thu, 15 Aug 2024 16:37:35 +0200 Subject: [PATCH] Fixed a bug that prevented storing passwords for new cctray pipelines. This should fix / help with #13. --- CCMenu/Resources/Info.plist | 4 ++-- .../Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CCMenu/Resources/Info.plist b/CCMenu/Resources/Info.plist index e72a372..077a826 100644 --- a/CCMenu/Resources/Info.plist +++ b/CCMenu/Resources/Info.plist @@ -19,7 +19,7 @@ CFBundlePackageType $(PRODUCT_BUNDLE_PACKAGE_TYPE) CFBundleShortVersionString - 25.0 + 25.1 CFBundleURLTypes @@ -34,7 +34,7 @@ CFBundleVersion - 2250.1 + 2251.1 LSApplicationCategoryType public.app-category.developer-tools LSMinimumSystemVersion diff --git a/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift b/CCMenu/Source/Pipeline Window/CCTray Sheets/CCTrayPipelineBuilder.swift index 8561fca..1aa5683 100644 --- 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... }