Add error codes to site breakage reports

Sources/PrivacyDashboard/WebsiteBreakage/WebsiteBreakage.swift

@@ -52,6 +52,8 @@ public struct WebsiteBreakage {
     let reportFlow: Source
     let protectionsState: Bool
     var lastSentDay: String?
+    let error: Error?
+    let httpStatusCode: Int?
 #if os(iOS)
     let siteType: SiteType // ?? not in documentation
     let atb: String
@@ -73,7 +75,9 @@ public struct WebsiteBreakage {
         ampURL: String,
         urlParametersRemoved: Bool,
         protectionsState: Bool,
-        reportFlow: Source
+        reportFlow: Source,
+        error: Error?,
+        httpStatusCode: Int?
     ) {
         self.siteUrl = siteUrl
         self.category = category
@@ -89,6 +93,8 @@ public struct WebsiteBreakage {
         self.protectionsState = protectionsState
         self.urlParametersRemoved = urlParametersRemoved
         self.reportFlow = reportFlow
+        self.error = error
+        self.httpStatusCode = httpStatusCode
     }
 #endif
 
@@ -110,7 +116,9 @@ public struct WebsiteBreakage {
         reportFlow: Source,
         siteType: SiteType,
         atb: String,
-        model: String
+        model: String,
+        error: Error?,
+        httpStatusCode: Int?
     ) {
         self.siteUrl = siteUrl
         self.category = category
@@ -129,12 +137,14 @@ public struct WebsiteBreakage {
         self.siteType = siteType
         self.atb = atb
         self.model = model
+        self.error = error
+        self.httpStatusCode = httpStatusCode
     }
 #endif
 
     /// A dictionary containing all the parameters needed from the Report Broken Site Pixel
-    public var requestParameters: [String: String] {
-        var result = [
+    public var requestParameters: [String: Codable] {
+        var result: [String: Codable] = [
             "siteUrl": siteUrl.trimmingQueryItemsAndFragment().absoluteString,
             "category": category,
             "description": description ?? "",
@@ -155,6 +165,15 @@ public struct WebsiteBreakage {
             result["lastSentDay"] = lastSentDay
         }
 
+        if let httpStatusCode {
+            result["httpErrorCode"] = String(httpStatusCode)
+        }
+
+        if let error = error as NSError? {
+            let errorDescription = "\(error.code) - \(error.localizedDescription)"
+            result["errorDescriptions"] = [errorDescription]
+        }
+
 #if os(iOS)
         result["siteType"] = siteType.rawValue
         result["atb"] = atb

Sources/PrivacyDashboard/WebsiteBreakage/WebsiteBreakageReporter.swift

@@ -37,7 +37,7 @@ public enum WebsiteBreakageReporterError: Error {
 public class WebsiteBreakageReporter {
 
     /// A closure that receives the Pixel's parameters
-    public typealias PixelHandler = (_ parameters: [String: String]) -> Void
+    public typealias PixelHandler = (_ parameters: [String: Codable]) -> Void
     /// Pixels are sent by the main apps not by BSK, this is the closure called by the class when a pixel need to be sent
     let pixelHandler: PixelHandler
     let persistencyManager: ExpiryStorage

Tests/PrivacyDashboardTests/WebsiteBreakageMoks.swift

@@ -35,7 +35,9 @@ struct WebsiteBreakageMoks {
                         ampURL: "test",
                         urlParametersRemoved: true,
                         protectionsState: true,
-                        reportFlow: .appMenu)
+                        reportFlow: .appMenu,
+                        error: nil,
+                        httpErrorCode: "")
     }
 
     static var testBreakage2: WebsiteBreakage {
@@ -52,7 +54,9 @@ struct WebsiteBreakageMoks {
                         ampURL: "test",
                         urlParametersRemoved: true,
                         protectionsState: true,
-                        reportFlow: .appMenu)
+                        reportFlow: .appMenu,
+                        error: nil,
+                        httpErrorCode: "")
     }
 
     static var testBreakage3: WebsiteBreakage {
@@ -69,6 +73,8 @@ struct WebsiteBreakageMoks {
                         ampURL: "test",
                         urlParametersRemoved: true,
                         protectionsState: true,
-                        reportFlow: .appMenu)
+                        reportFlow: .appMenu,
+                        error: nil,
+                        httpErrorCode: "")
     }
 }