Merge pull request #50 from mailchimp/fix/49

dkotter · web-flow · commit 5ef2ddc3f339 · 2024-08-20T08:33:28.000-06:00
Make the API authentication error message more general.
diff --git a/lib/mailchimp/mailchimp.php b/lib/mailchimp/mailchimp.php
@@ -128,6 +128,7 @@ public function get( $endpoint, $count = 10, $fields = array() ) {
 			// Check if Access Token is invalid/revoked.
 			if ( in_array( $request['response']['code'], array( 401, 403 ), true ) ) {
 				update_option( 'mailchimp_sf_auth_error', true );
+				return new WP_Error( 'mailchimp-auth-error', esc_html__( 'Authentication failed.', 'mailchimp' ) );
 			}
 
 			$error = json_decode( $request['body'], true );

Original file line number	Diff line number	Diff line change
`@@ -128,6 +128,7 @@ public function get( $endpoint, $count = 10, $fields = array() ) {`
`128`	`128`	`// Check if Access Token is invalid/revoked.`
`129`	`129`	`if ( in_array( $request['response']['code'], array( 401, 403 ), true ) ) {`
`130`	`130`	`update_option( 'mailchimp_sf_auth_error', true );`
	`131`	`+ return new WP_Error( 'mailchimp-auth-error', esc_html__( 'Authentication failed.', 'mailchimp' ) );`
`131`	`132`	`}`
`132`	`133`
`133`	`134`	`$error = json_decode( $request['body'], true );`