Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Commit 38e5181

Browse files
authored
Remove malformed error pixel (#3612)
Task/Issue URL: https://app.asana.com/0/414235014887631/1208683003138954/f Tech Design URL: CC: Description: This PR removes the malformed error pixel.
1 parent ce28078 commit 38e5181

File tree

8 files changed

+9
-21
lines changed

8 files changed

+9
-21
lines changed

DuckDuckGo.xcodeproj/project.pbxproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15347,7 +15347,7 @@
1534715347
repositoryURL = "https://github.com/duckduckgo/BrowserServicesKit";
1534815348
requirement = {
1534915349
kind = exactVersion;
15350-
version = 215.0.1;
15350+
version = 216.0.0;
1535115351
};
1535215352
};
1535315353
9FF521422BAA8FF300B9819B /* XCRemoteSwiftPackageReference "lottie-spm" */ = {

DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

DuckDuckGo/NetworkProtection/AppAndExtensionAndAgentTargets/NetworkProtectionPixelEvent.swift

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,6 @@ enum NetworkProtectionPixelEvent: PixelKitEventV2 {
113113
case networkProtectionConfigurationErrorLoadingCachedConfig(_ error: Error)
114114
case networkProtectionConfigurationFailedToParse(_ error: Error)
115115

116-
case networkProtectionMalformedErrorDetected(_ error: Error)
117-
118116
case networkProtectionUnhandledError(function: String, line: Int, error: Error)
119117

120118
/// Name of the pixel event
@@ -330,9 +328,6 @@ enum NetworkProtectionPixelEvent: PixelKitEventV2 {
330328
case .networkProtectionConfigurationFailedToParse:
331329
return "netp_ev_configuration_failed_to_parse"
332330

333-
case .networkProtectionMalformedErrorDetected:
334-
return "netp_malformed_error_detected"
335-
336331
case .networkProtectionUnhandledError:
337332
return "netp_unhandled_error"
338333
}
@@ -445,8 +440,7 @@ enum NetworkProtectionPixelEvent: PixelKitEventV2 {
445440
.networkProtectionServerMigrationSuccess,
446441
.networkProtectionDNSUpdateCustom,
447442
.networkProtectionDNSUpdateDefault,
448-
.networkProtectionConfigurationInvalidPayload,
449-
.networkProtectionMalformedErrorDetected:
443+
.networkProtectionConfigurationInvalidPayload:
450444
return nil
451445
}
452446
}
@@ -473,8 +467,7 @@ enum NetworkProtectionPixelEvent: PixelKitEventV2 {
473467
.networkProtectionSystemExtensionActivationFailure(let error),
474468
.networkProtectionServerMigrationFailure(let error),
475469
.networkProtectionConfigurationErrorLoadingCachedConfig(let error),
476-
.networkProtectionConfigurationFailedToParse(let error),
477-
.networkProtectionMalformedErrorDetected(let error):
470+
.networkProtectionConfigurationFailedToParse(let error):
478471
return error
479472
case .networkProtectionActiveUser,
480473
.networkProtectionNewUser,

DuckDuckGo/NetworkProtection/NetworkExtensionTargets/NetworkExtensionTargets/MacPacketTunnelProvider.swift

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -390,11 +390,6 @@ final class MacPacketTunnelProvider: PacketTunnelProvider {
390390
NetworkProtectionPixelEvent.networkProtectionTunnelStartAttemptOnDemandWithoutAccessToken,
391391
frequency: .legacyDailyAndCount,
392392
includeAppVersionParameter: true)
393-
case .malformedErrorDetected(let error):
394-
PixelKit.fire(
395-
NetworkProtectionPixelEvent.networkProtectionMalformedErrorDetected(error),
396-
frequency: .legacyDailyAndCount,
397-
includeAppVersionParameter: true)
398393
}
399394
}
400395

LocalPackages/DataBrokerProtection/Package.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ let package = Package(
2929
targets: ["DataBrokerProtection"])
3030
],
3131
dependencies: [
32-
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "215.0.1"),
32+
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "216.0.0"),
3333
.package(path: "../SwiftUIExtensions"),
3434
.package(path: "../AppKitExtensions"),
3535
.package(path: "../XPCHelper"),

LocalPackages/FeatureFlags/Package.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ let package = Package(
3232
targets: ["FeatureFlags"]),
3333
],
3434
dependencies: [
35-
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "215.0.1"),
35+
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "216.0.0"),
3636
],
3737
targets: [
3838
// Targets are the basic building blocks of a package, defining a module or a test suite.

LocalPackages/NetworkProtectionMac/Package.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ let package = Package(
3232
.library(name: "VPNAppLauncher", targets: ["VPNAppLauncher"]),
3333
],
3434
dependencies: [
35-
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "215.0.1"),
35+
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "216.0.0"),
3636
.package(url: "https://github.com/airbnb/lottie-spm", exact: "4.4.3"),
3737
.package(path: "../AppLauncher"),
3838
.package(path: "../UDSHelper"),

LocalPackages/SubscriptionUI/Package.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ let package = Package(
1212
targets: ["SubscriptionUI"]),
1313
],
1414
dependencies: [
15-
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "215.0.1"),
15+
.package(url: "https://github.com/duckduckgo/BrowserServicesKit", exact: "216.0.0"),
1616
.package(path: "../SwiftUIExtensions"),
1717
.package(path: "../FeatureFlags")
1818
],

0 commit comments

Comments
 (0)