From 11eb6e0a9301a16258fb648ed7cb92688ed43279 Mon Sep 17 00:00:00 2001
From: Krisjanis Gaidis <kgaidis@stripe.com>
Date: Tue, 25 Oct 2022 12:26:03 -0400
Subject: [PATCH] Financial Connections: fixed/improved synchronize API all
 parameters.

---
 .../Source/API Bindings/APIVersion.swift       |  2 +-
 .../FinancialConnectionsAPIClient.swift        | 18 ++++++++++--------
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/APIVersion.swift b/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/APIVersion.swift
index 78e2928ab74..3add7c5a826 100644
--- a/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/APIVersion.swift	
+++ b/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/APIVersion.swift	
@@ -15,7 +15,7 @@ struct APIVersion {
 
      - Note: Update this value when a new API version is ready for use in production.
      */
-    private static let apiVersion: Int = 1
+    static let apiVersion: Int = 1 // WARNING: this is also referenced in other places, so double check changes!
     private static let header = "financial_connections_client_api_beta=v\(apiVersion)"
 
     static func configureFinancialConnectionsAPIVersion(apiClient: STPAPIClient) {
diff --git a/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/FinancialConnectionsAPIClient.swift b/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/FinancialConnectionsAPIClient.swift
index b22a6a9fdb5..ed0bd65bf80 100644
--- a/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/FinancialConnectionsAPIClient.swift	
+++ b/StripeFinancialConnections/StripeFinancialConnections/Source/API Bindings/FinancialConnectionsAPIClient.swift	
@@ -80,16 +80,18 @@ extension STPAPIClient: FinancialConnectionsAPIClient {
     func generateSessionManifest(clientSecret: String, returnURL: String?) -> Promise<FinancialConnectionsSynchronize> {
         let parameters: [String: Any] = [
             "client_secret": clientSecret,
-            "mobile" : [
-                "sdk_type": "ios",
-                "fullscreen": true,
-                "hide_close_button": true,
-                "sdk_version": 1,
-            ],
+            "mobile": {
+                var mobileParameters: [String:Any] = [
+                    "sdk_type": "ios",
+                    "fullscreen": true,
+                    "hide_close_button": true,
+                    "sdk_version": APIVersion.apiVersion,
+                ]
+                mobileParameters["app_return_url"] = returnURL
+                return mobileParameters
+            }(),
             "locale": Locale.current.identifier,
         ]
-        // parameters["app_return_url"] = returnURL // TODO(kgaidis): double-check these parameters
-        
         return self.post(
             resource: "financial_connections/sessions/synchronize",
             parameters: parameters