Release 5.4.4 (#260)

## [5.4.4] - 2022-01-19
### Fixed
- Fixed memory leak when stream connections are terminated by updating `LDSwiftEventSource` dependency to [1.3.0](https://github.com/launchdarkly/swift-eventsource/releases/tag/1.3.0).
- The SDK would not allow additional fields on `delete` flag stream events. This has been updated to allow additional fields for improved future compatibility.
- Improved internal `Throttler` implementation to reduce concurrency concerns.
- Removed unneeded `Cartfile` definining `LDSwiftEventSource` dependency, which when bundled could lead to warning messages that `LDSwiftEventSource` definitions are implemented in multiple frameworks.

Author: louis-launchdarkly

.circleci/config.yml

@@ -8,7 +8,7 @@ jobs:
         type: string
       ssh-fix:
         type: boolean
-        default: true
+        default: false
       build-doc:
         type: boolean
         default: false
@@ -24,19 +24,15 @@ jobs:
     steps:
     - checkout
 
-    # This hack shouldn't be necessary, as we don't actually use SSH
-    # to get any dependencies, but for some reason starting in the
-    # '12.0.0' Xcode image it's become necessary.
+    # There's an XCode bug present in the 12.0.1 CircleCI image that prevents fetching SSH
+    # dependencies from working in some cases, so we disable CircleCI's rewriting of the HTTPS
+    # GitHub URLs to SSH.
     - when:
         condition: <<parameters.ssh-fix>>
         steps:
           - run:
-              name: SSH fingerprint fix
-              command: |
-                sudo defaults write com.apple.dt.Xcode IDEPackageSupportUseBuiltinSCM YES
-                rm ~/.ssh/id_rsa || true
-                for ip in $(dig @8.8.8.8 bitbucket.org +short); do ssh-keyscan bitbucket.org,$ip; ssh-keyscan $ip; done 2>/dev/null >> ~/.ssh/known_hosts || true
-                for ip in $(dig @8.8.8.8 github.com +short); do ssh-keyscan github.com,$ip; ssh-keyscan $ip; done 2>/dev/null >> ~/.ssh/known_hosts || true
+              name: SSH fix
+              command: git config --global --unset url.ssh://git@github.com.insteadof
 
     - run:
         name: Setup for builds
@@ -87,11 +83,25 @@ jobs:
     - when:
         condition: <<parameters.build-doc>>
         steps:
+          - restore_cache:
+              key: v1-gem-cache-<<parameters.xcode-version>>-
+
           - run:
-              name: Build Documentation
+              name: Install jazzy gem
               command: |
-                sudo gem install jazzy
-                jazzy -o artifacts/docs
+                gem install jazzy
+                gem cleanup
+                # Used as cache key to prevent storing redundant caches
+                gem list > /tmp/cache-key.txt
+
+          - save_cache:
+              key: v1-gem-cache-<<parameters.xcode-version>>-{{ checksum "/tmp/cache-key.txt" }}
+              paths:
+                - ~/.gem
+
+          - run:
+              name: Build Documentation
+              command: jazzy -o artifacts/docs
 
     - when:
         condition: <<parameters.run-lint>>
@@ -116,18 +126,22 @@ workflows:
 
   build:
     jobs:
+      - build:
+          name: Xcode 13.1 - Swift 5.5
+          xcode-version: '13.1.0'
+          ios-sim: 'platform=iOS Simulator,name=iPhone 11,OS=15.0'
+          build-doc: true
+          run-lint: true
       - build:
           name: Xcode 12.5 - Swift 5.4
           xcode-version: '12.5.0'
           ios-sim: 'platform=iOS Simulator,name=iPhone 8,OS=14.5'
-          build-doc: true
-          run-lint: true
       - build:
           name: Xcode 12.0 - Swift 5.3
           xcode-version: '12.0.1'
           ios-sim: 'platform=iOS Simulator,name=iPhone 8,OS=14.0'
+          ssh-fix: true
       - build:
           name: Xcode 11.4 - Swift 5.2
           xcode-version: '11.4.1'
           ios-sim: 'platform=iOS Simulator,name=iPhone 8,OS=12.2'
-          ssh-fix: false

CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to the LaunchDarkly iOS SDK will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org).
 
+## [5.4.4] - 2022-01-19
+### Fixed
+- Fixed memory leak when stream connections are terminated by updating `LDSwiftEventSource` dependency to [1.3.0](https://github.com/launchdarkly/swift-eventsource/releases/tag/1.3.0).
+- The SDK would not allow additional fields on `delete` flag stream events. This has been updated to allow additional fields for improved future compatibility.
+- Improved internal `Throttler` implementation to reduce concurrency concerns.
+- Removed unneeded `Cartfile` definining `LDSwiftEventSource` dependency, which when bundled could lead to warning messages that `LDSwiftEventSource` definitions are implemented in multiple frameworks.
+
 ## [5.4.3] - 2021-08-13
 ### Fixed
 - Fixed an issue where `304 NOT_MODIFIED` responses to SDK polling mode requests would be considered error responses. This could cause the completion on a `identify` request to not complete, and gave erroneous connection information data and logging output. 

Cartfile

@@ -2,7 +2,7 @@
 Pod::Spec.new do |ld|
 
   ld.name         = "LaunchDarkly"
-  ld.version      = "5.4.3"
+  ld.version      = "5.4.4"
   ld.summary      = "iOS SDK for LaunchDarkly"
 
   ld.description  = <<-DESC
@@ -35,6 +35,6 @@ Pod::Spec.new do |ld|
   ld.swift_version = '5.0'
 
   ld.subspec 'Core' do |es|
-    es.dependency 'LDSwiftEventSource', '1.2.1'
+    es.dependency 'LDSwiftEventSource', '1.3.0'
   end
 end

LaunchDarkly.podspec

@@ -1583,7 +1583,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				MODULEMAP_FILE = "";
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-tvOS";
 				PRODUCT_NAME = LaunchDarkly_tvOS;
@@ -1606,7 +1606,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				MODULEMAP_FILE = "";
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-tvOS";
 				PRODUCT_NAME = LaunchDarkly_tvOS;
@@ -1629,7 +1629,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-macOS";
 				PRODUCT_NAME = LaunchDarkly_macOS;
 				SDKROOT = macosx;
@@ -1650,7 +1650,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-macOS";
 				PRODUCT_NAME = LaunchDarkly_macOS;
 				SDKROOT = macosx;
@@ -1694,7 +1694,7 @@
 				CURRENT_PROJECT_VERSION = 1;
 				DEBUG_INFORMATION_FORMAT = dwarf;
 				DYLIB_COMPATIBILITY_VERSION = 5.4.0;
-				DYLIB_CURRENT_VERSION = 5.4.3;
+				DYLIB_CURRENT_VERSION = 5.4.4;
 				ENABLE_STRICT_OBJC_MSGSEND = YES;
 				ENABLE_TESTABILITY = YES;
 				FRAMEWORK_VERSION = B;
@@ -1765,7 +1765,7 @@
 				CURRENT_PROJECT_VERSION = 1;
 				DEBUG_INFORMATION_FORMAT = "dwarf-with-dsym";
 				DYLIB_COMPATIBILITY_VERSION = 5.4.0;
-				DYLIB_CURRENT_VERSION = 5.4.3;
+				DYLIB_CURRENT_VERSION = 5.4.4;
 				ENABLE_NS_ASSERTIONS = NO;
 				ENABLE_STRICT_OBJC_MSGSEND = YES;
 				FRAMEWORK_VERSION = B;
@@ -1804,7 +1804,7 @@
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
 				LD_DYLIB_INSTALL_NAME = "$(DYLIB_INSTALL_NAME_BASE:standardizepath)/$(EXECUTABLE_PATH)";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				MODULEMAP_FILE = "$(PROJECT_DIR)/Framework/module.modulemap";
 				PRODUCT_BUNDLE_IDENTIFIER = com.launchdarkly.Darkly;
 				PRODUCT_NAME = LaunchDarkly;
@@ -1824,7 +1824,7 @@
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
 				LD_DYLIB_INSTALL_NAME = "$(DYLIB_INSTALL_NAME_BASE:standardizepath)/$(EXECUTABLE_PATH)";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				MODULEMAP_FILE = "$(PROJECT_DIR)/Framework/module.modulemap";
 				PRODUCT_BUNDLE_IDENTIFIER = com.launchdarkly.Darkly;
 				PRODUCT_NAME = LaunchDarkly;
@@ -1866,7 +1866,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-watchOS";
 				PRODUCT_NAME = LaunchDarkly_watchOS;
 				SDKROOT = watchos;
@@ -1888,7 +1888,7 @@
 				GCC_C_LANGUAGE_STANDARD = gnu11;
 				INFOPLIST_FILE = "$(PROJECT_DIR)/LaunchDarkly/LaunchDarkly/Support/Info.plist";
 				INSTALL_PATH = "$(LOCAL_LIBRARY_DIR)/Frameworks";
-				MARKETING_VERSION = 5.4.3;
+				MARKETING_VERSION = 5.4.4;
 				PRODUCT_BUNDLE_IDENTIFIER = "com.launchdarkly.Darkly-watchOS";
 				PRODUCT_NAME = LaunchDarkly_watchOS;
 				SDKROOT = watchos;
@@ -1962,7 +1962,7 @@
 			repositoryURL = "https://github.com/LaunchDarkly/swift-eventsource.git";
 			requirement = {
 				kind = exactVersion;
-				version = 1.2.1;
+				version = 1.3.0;
 			};
 		};
 		B4903D9624BD61B200F087C4 /* XCRemoteSwiftPackageReference "OHHTTPStubs" */ = {
@@ -1978,15 +1978,15 @@
 			repositoryURL = "https://github.com/Quick/Nimble.git";
 			requirement = {
 				kind = exactVersion;
-				version = 9.2.0;
+				version = 9.2.1;
 			};
 		};
 		B4903D9C24BD61EF00F087C4 /* XCRemoteSwiftPackageReference "Quick" */ = {
 			isa = XCRemoteSwiftPackageReference;
 			repositoryURL = "https://github.com/Quick/Quick.git";
 			requirement = {
 				kind = exactVersion;
-				version = 3.1.2;
+				version = 4.0.0;
 			};
 		};
 /* End XCRemoteSwiftPackageReference section */

LaunchDarkly.xcodeproj/project.pbxproj

@@ -9,7 +9,7 @@ import Foundation
 
 struct ErrorObserver {
     weak var owner: LDObserverOwner?
-    var errorHandler: LDErrorHandler?
+    let errorHandler: LDErrorHandler
 
     init(owner: LDObserverOwner, errorHandler: @escaping LDErrorHandler) {
         self.owner = owner

LaunchDarkly/LaunchDarkly/Models/ErrorObserver.swift

@@ -47,7 +47,7 @@ class DiagnosticReporter: DiagnosticReporting {
                     sendDiagnosticEventAsync(diagnosticEvent: initEvent)
                 }
 
-                timer = LDTimer(withTimeInterval: service.config.diagnosticRecordingInterval, repeats: true, fireQueue: workQueue) {
+                timer = LDTimer(withTimeInterval: service.config.diagnosticRecordingInterval, fireQueue: workQueue) {
                     self.sendDiagnosticEventSync(diagnosticEvent: cache.getCurrentStatsAndReset())
                 }
             }

LaunchDarkly/LaunchDarkly/ServiceObjects/DiagnosticReporter.swift

@@ -135,7 +135,7 @@ struct EnvironmentReporter: EnvironmentReporting {
     var shouldThrottleOnlineCalls: Bool { true }
     #endif
 
-    let sdkVersion = "5.4.3"
+    let sdkVersion = "5.4.4"
     // Unfortunately, the following does not function in certain configurations, such as when included through SPM
 //    var sdkVersion: String {
 //        Bundle(for: LDClient.self).infoDictionary?["CFBundleShortVersionString"] as? String ?? "5.x"

LaunchDarkly/LaunchDarkly/ServiceObjects/EnvironmentReporter.swift

@@ -22,27 +22,15 @@ final class ErrorNotifier: ErrorNotifying {
     }
 
     func removeObservers(for owner: LDObserverOwner) {
-        errorObservers = errorObservers.filter { $0.owner !== owner }
+        errorObservers.removeAll { $0.owner === owner }
     }
 
     func notifyObservers(of error: Error) {
         removeOldObservers()
-        errorObservers.forEach { $0.errorHandler?(error) }
+        errorObservers.forEach { $0.errorHandler(error) }
     }
 
     private func removeOldObservers() {
         errorObservers = errorObservers.filter { $0.owner != nil }
     }
 }
-
-#if DEBUG
-extension ErrorNotifier {
-    func erase(owner: LDObserverOwner) {
-        for index in 0..<errorObservers.count {
-            guard errorObservers[index].owner === owner
-            else { continue }
-            errorObservers[index].owner = nil
-        }
-    }
-}
-#endif

LaunchDarkly/LaunchDarkly/ServiceObjects/ErrorNotifier.swift

@@ -97,7 +97,7 @@ class EventReporter: EventReporting {
     private func startReporting(isOnline: Bool) {
         guard isOnline && !isReportingActive
         else { return }
-        eventReportTimer = LDTimer(withTimeInterval: service.config.eventFlushInterval, repeats: true, fireQueue: eventQueue, execute: reportEvents)
+        eventReportTimer = LDTimer(withTimeInterval: service.config.eventFlushInterval, fireQueue: eventQueue, execute: reportEvents)
     }
 
     private func stopReporting() {