Merge pull request #522 from checkout/feature/pass-correlation-id-to-…

…risk-sdk

Pass correlation ID to risk SDK

.github/workflows/codeql-analysis.yml

@@ -19,10 +19,17 @@ concurrency:
   group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
   cancel-in-progress: true
 
+env:
+  destination: "platform=iOS Simulator,name=iPhone 15 Pro,OS=latest"
+  configuration: "Debug"
+  noIndex: "COMPILER_INDEX_STORE_ENABLE=NO"
+  noSigning: "CODE_SIGNING_ALLOWED=NO"
+  versionXcode: "15.2"
+
 jobs:
   analyze:
     name: Analyze
-    runs-on: [ macos-latest ]
+    runs-on: [ macos-13-large ]
     permissions:
       actions: read
       contents: read
@@ -45,24 +52,37 @@ jobs:
       with:
         languages: ${{ matrix.language }}
         queries: security-and-quality
-
+
+    - name: Select Xcode
+      run: |
+        sudo xcode-select -switch /Applications/Xcode_${versionXcode}.app
+
+    - name: Log xcodebuild Version
+      run: |
+        xcodebuild -version
+
+
     - name: Build Frames
       run: |
-        xcodebuild -scheme Frames -destination "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest"
+        xcodebuild -scheme Frames -destination "${destination}" "${noIndex}" "${noSigning}" | xcpretty
+
+    - name: Build CheckoutTests
+      run: |
+        xcodebuild build -scheme CheckoutTests -destination "${destination}" "${noIndex}" "${noSigning}" | xcpretty
 
     - name: Build FramesTests
       run: |
-        xcodebuild -scheme FramesTests -destination "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest" test
+        xcodebuild build -scheme FramesTests -destination "${destination}" "${noIndex}" "${noSigning}" | xcpretty
 
     - name: Build iOS Example Frame SPM
       run: |
         cd iOS\ Example\ Frame\ SPM
-        xcodebuild build -scheme  iOS\ Example\ Frame -destination "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest"
+        xcodebuild build -scheme  iOS\ Example\ Frame -destination "${destination}" "${noIndex}" "${noSigning}" | xcpretty
 
     - name: Build UITest
       run: |
         cd iOS\ Example\ Frame\ SPM
-        xcodebuild -scheme UITest -destination "platform=iOS Simulator,name=iPhone 14 Pro,OS=latest"
+        xcodebuild -scheme UITest -destination "${destination}" "${noIndex}" "${noSigning}" | xcpretty
 
     # Perform analysis on the code
     - name: Analyze code with CodeQL

.swiftpm/xcode/xcshareddata/xcschemes/CheckoutTests.xcscheme

@@ -8,7 +8,7 @@
       <BuildActionEntries>
          <BuildActionEntry
             buildForTesting = "YES"
-            buildForRunning = "NO"
+            buildForRunning = "YES"
             buildForProfiling = "NO"
             buildForArchiving = "NO"
             buildForAnalyzing = "NO">
@@ -53,6 +53,15 @@
       savedToolIdentifier = ""
       useCustomWorkingDirectory = "NO"
       debugDocumentVersioning = "YES">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "CheckoutTests"
+            BuildableName = "CheckoutTests"
+            BlueprintName = "CheckoutTests"
+            ReferencedContainer = "container:">
+         </BuildableReference>
+      </MacroExpansion>
    </ProfileAction>
    <AnalyzeAction
       buildConfiguration = "Debug">

.swiftpm/xcode/xcshareddata/xcschemes/FramesTests.xcscheme

@@ -5,6 +5,22 @@
    <BuildAction
       parallelizeBuildables = "YES"
       buildImplicitDependencies = "YES">
+      <BuildActionEntries>
+         <BuildActionEntry
+            buildForTesting = "YES"
+            buildForRunning = "YES"
+            buildForProfiling = "NO"
+            buildForArchiving = "NO"
+            buildForAnalyzing = "NO">
+            <BuildableReference
+               BuildableIdentifier = "primary"
+               BlueprintIdentifier = "FramesTests"
+               BuildableName = "FramesTests"
+               BlueprintName = "FramesTests"
+               ReferencedContainer = "container:">
+            </BuildableReference>
+         </BuildActionEntry>
+      </BuildActionEntries>
    </BuildAction>
    <TestAction
       buildConfiguration = "Debug"
@@ -42,6 +58,15 @@
       savedToolIdentifier = ""
       useCustomWorkingDirectory = "NO"
       debugDocumentVersioning = "YES">
+      <MacroExpansion>
+         <BuildableReference
+            BuildableIdentifier = "primary"
+            BlueprintIdentifier = "FramesTests"
+            BuildableName = "FramesTests"
+            BlueprintName = "FramesTests"
+            ReferencedContainer = "container:">
+         </BuildableReference>
+      </MacroExpansion>
    </ProfileAction>
    <AnalyzeAction
       buildConfiguration = "Debug">

Checkout.podspec

@@ -20,6 +20,6 @@ Pod::Spec.new do |s|
   s.exclude_files = "Checkout/Samples/**"
 
   s.dependency 'CheckoutEventLoggerKit', '~> 1.2.4'
-  s.dependency 'Risk', '2.0.1'
+  s.dependency 'Risk', '2.0.3'
 
 end

Checkout/Samples/CocoapodsSample/Podfile

@@ -5,6 +5,7 @@ target 'CheckoutCocoapodsSample' do
   use_frameworks!
 
   # Pods for CheckoutSDKCocoapodsSample
-  pod 'Checkout', '4.3.2'
+  # pod 'Checkout', '4.3.2'
+  pod 'Frames', :git => 'https://github.com/checkout/frames-ios', :branch => 'feature/pass-correlation-id-to-risk-sdk'
 
 end

Checkout/Samples/SPMSample/CheckoutSPMSample.xcodeproj/project.pbxproj

@@ -511,8 +511,8 @@
 			isa = XCRemoteSwiftPackageReference;
 			repositoryURL = "https://github.com/checkout/frames-ios";
 			requirement = {
-				kind = exactVersion;
-				version = 4.3.2;
+				branch = "feature/pass-correlation-id-to-risk-sdk";
+				kind = branch;
 			};
 		};
 /* End XCRemoteSwiftPackageReference section */

Checkout/Source/Tokenisation/CheckoutAPIService.swift

@@ -59,7 +59,7 @@ final public class CheckoutAPIService: CheckoutAPIProtocol {
       riskEnvironment = .sandbox
     }
 
-    let riskConfig = RiskConfig(publicKey: publicKey, environment: riskEnvironment, framesMode: true)
+    let riskConfig = RiskConfig(publicKey: publicKey, environment: riskEnvironment, framesMode: true, correlationId: logManager.correlationID)
     let riskSDK = Risk.init(config: riskConfig)
 
     logManager.setup(

CheckoutTests/Integration/CheckoutAPIServiceIntegrationTests.swift

@@ -66,7 +66,7 @@ final class CheckoutAPIServiceIntegrationTests: XCTestCase {
 
     // details associated with default apple pay token
     let expectedApplePayDetails = ApplePayDetails(
-      expiryDate: try! CardValidator(environment: .sandbox).validate(expiryMonth: 3, expiryYear: 24).get(),
+      expiryDate: .init(month: 3, year: 2024),
       bin: "537426",
       last4: "7789"
     )

Package.resolved

@@ -14,8 +14,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/checkout/checkout-risk-sdk-ios.git",
       "state" : {
-        "revision" : "4823f05166dca8392a41b56b975515c7e0f1a8da",
-        "version" : "2.0.1"
+        "revision" : "a5df46ecd4324661459faa9e9ac18d627b4d26aa",
+        "version" : "2.0.3"
       }
     },
     {

Package.swift

@@ -20,7 +20,7 @@ let package = Package(
             exact: "3.5.9"),
         .package(
             url: "https://github.com/checkout/checkout-risk-sdk-ios.git",
-            exact: "2.0.1"),
+            exact: "2.0.3"),
         .package(
             url: "https://github.com/checkout/checkout-event-logger-ios-framework.git",
             from: "1.2.4"

iOS Example Frame SPM/iOS Example Frame SPM.xcodeproj/project.pbxproj

@@ -1238,8 +1238,8 @@
 			isa = XCRemoteSwiftPackageReference;
 			repositoryURL = "https://github.com/checkout/frames-ios";
 			requirement = {
-				kind = exactVersion;
-				version = 4.3.2;
+				branch = "feature/pass-correlation-id-to-risk-sdk";
+				kind = branch;
 			};
 		};
 		16C3F83E2A7927ED00690639 /* XCRemoteSwiftPackageReference "swift-snapshot-testing" */ = {

iOS Example Frame/Podfile

@@ -6,8 +6,9 @@ target 'iOS Example Frame' do
   use_frameworks!
 
   # Pods for iOS Example Custom
-  pod 'Frames', '4.3.2'
-
+  # pod 'Frames', '4.3.2'
+  pod 'Frames', :git => 'https://github.com/checkout/frames-ios', :branch => 'feature/pass-correlation-id-to-risk-sdk'
+
 end
 
 post_install do |installer|