Use default bundle policy (#25)

* Change bundle policy, specify minimum Infinity version

* Remove . and newline

* Fix RTCConfigurationDefaultTests

* Fix version comparison

* Move the warning to InfinityClientFactory

Docs/pexip-swift-sdk-docs/Sources/PexipSwiftSDK/PexipSwiftSDK.docc/Articles/Installation.md

@@ -8,6 +8,7 @@ Integrate Pexip Swift SDK into your project using Swift Package Manager, CocoaPo
 - macOS 10.15+
 - Swift 5.5 with structured concurrency support
 - Xcode 13
+- Pexip Infinity v29 and higher
 
 ## Swift Package Manager
 

README.md

@@ -38,6 +38,7 @@ media signaling technologies in the future, or Infinity might be interchanged wi
 - macOS 10.15+
 - Swift 5.5 with structured concurrency support
 - Xcode 13
+- Pexip Infinity v29 and higher
 
 :warning: **The project doesn't compile on Xcode 14.3 because of incorrect availability attributes CGDisplayStream.h file (introduced: is 13.0 instead of 10.8). Let's hope it's fixed in future Xcode versions.**
 

Sources/PexipInfinityClient/InfinityClientFactory.swift

@@ -109,6 +109,7 @@ public struct InfinityClientFactory {
         )
     }
 
+    // swiftlint:disable function_body_length
     /**
      Creates a new instance of ``Conference`` type.
 
@@ -124,6 +125,21 @@ public struct InfinityClientFactory {
         alias: ConferenceAlias,
         token: ConferenceToken
     ) -> Conference {
+        if token.version.versionId < "29" {
+            logger?.warn(
+                """
+
+                WARNING: Infinity v\(token.version.versionId) support.
+
+                We offer only limited support for Infinity versions prior to v29.
+                You may experience problems with sending and receiving presentation.
+                Use `presentationInMain` to mix presentation with main video feed.
+
+                This will be a fatar error in Q4 2023.
+                """
+            )
+        }
+
         let conferenceService = infinityService().node(url: node).conference(alias: alias)
         let tokenStore = TokenStore(token: token)
         let roster = roster(token: token, service: conferenceService)
@@ -164,6 +180,7 @@ public struct InfinityClientFactory {
             logger: logger
         )
     }
+    // swiftlint:enable function_body_length
 
     // MARK: - Internal methods
 

Sources/PexipRTC/Internal/Extensions/RTCConfiguration+Default.swift

@@ -1,5 +1,5 @@
 //
-// Copyright 2022 Pexip AS
+// Copyright 2022-2023 Pexip AS
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -29,7 +29,7 @@ extension RTCConfiguration {
                 credential: $0.password
             )
         }
-        configuration.bundlePolicy = .maxBundle
+        configuration.bundlePolicy = .balanced
         configuration.sdpSemantics = .unifiedPlan
         configuration.enableDscp = dscp
         configuration.continualGatheringPolicy = .gatherContinually

Tests/PexipRTCTests/Internal/Extensions/RTCConfigurationDefaultTests.swift

@@ -1,5 +1,5 @@
 //
-// Copyright 2022 Pexip AS
+// Copyright 2022-2023 Pexip AS
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -36,7 +36,7 @@ final class RTCConfigurationDefaultTests: XCTestCase {
         XCTAssertEqual(configuration.iceServers.count, 1)
         XCTAssertEqual(configuration.iceServers.first?.urlStrings, iceServer.urls)
         XCTAssertEqual(configuration.sdpSemantics, .unifiedPlan)
-        XCTAssertEqual(configuration.bundlePolicy, .maxBundle)
+        XCTAssertEqual(configuration.bundlePolicy, .balanced)
         XCTAssertEqual(configuration.continualGatheringPolicy, .gatherContinually)
         XCTAssertEqual(configuration.rtcpMuxPolicy, .require)
         XCTAssertEqual(configuration.tcpCandidatePolicy, .enabled)