fix: bump Appsero\client to v2.0.4

justlevine · justlevine · commit 29cfbf416c3a · 2024-11-30T16:00:19.000+02:00
diff --git a/composer.json b/composer.json
@@ -25,7 +25,7 @@
 		"php": "^7.1 || ^8.0",
 		"webonyx/graphql-php": "14.11.10",
 		"ivome/graphql-relay-php": "0.7.0",
-		"appsero/client": "1.2.1"
+		"appsero/client": "2.0.4"
 	},
 	"require-dev": {
 		"automattic/vipwpcs": "^3.0",
diff --git a/composer.lock b/composer.lock
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -38,5 +38,4 @@ parameters:
     excludePaths:
       analyseAndScan:
         - */node_modules/*
-        - */vendor/*
     ignoreErrors:
diff --git a/wp-graphql.php b/wp-graphql.php
@@ -167,17 +167,28 @@ function graphql_init_appsero_telemetry() {
 		return;
 	}
 
-	$client   = new Appsero\Client( 'cd0d1172-95a0-4460-a36a-2c303807c9ef', 'WPGraphQL', __FILE__ );
-	$insights = $client->insights();
+	// Wrap the Appsero client in a try/catch block to prevent fatal errors
+	try {
+		$client = new \Appsero\Client( 'cd0d1172-95a0-4460-a36a-2c303807c9ef', 'WPGraphQL', __FILE__ );
 
-	// If the Appsero client has the add_plugin_data method, use it
-	if ( method_exists( $insights, 'add_plugin_data' ) ) {
-		// @phpstan-ignore-next-line
-		$insights->add_plugin_data();
-	}
+		/** @var \Appsero\Insights $insights */
+		$insights = $client->insights();
+
+		// If the Appsero client has the add_plugin_data method, use it
+		if ( method_exists( $insights, 'add_plugin_data' ) ) {
+			$insights->add_plugin_data();
+		}
 
-	// @phpstan-ignore-next-line
-	$insights->init();
+		$insights->init();
+	} catch ( \Throwable $e ) {
+		error_log( // phpcs:ignore WordPress.PHP.DevelopmentFunctions.error_log_error_log -- Error logging is intentional here.
+			sprintf(
+			// translators: %s is the error message
+				__( 'Error initializing Appsero: %s', 'wp-graphql' ),
+				$e->getMessage()
+			)
+		);
+	}
 }
 
 graphql_init_appsero_telemetry();
