Add logging for when we receive a 401 (#4337)

Co-authored-by: daledupreez <[email protected]>

Author: 2 people

includes/class-wc-stripe-api.php

@@ -247,7 +247,7 @@ public static function request( $request, $api = 'charges', $method = 'POST', $w
 	public static function retrieve( $api ) {
 		// If we have an option flag indicating that the secret key is not valid, we don't attempt the API call and we return an error.
 		$invalid_api_keys_option_key = WC_Stripe_Mode::is_test() ? self::TEST_MODE_INVALID_API_KEYS_OPTION_KEY : self::LIVE_MODE_INVALID_API_KEYS_OPTION_KEY;
-		$invalid_api_keys_detected = get_option( $invalid_api_keys_option_key );
+		$invalid_api_keys_detected   = get_option( $invalid_api_keys_option_key );
 		if ( $invalid_api_keys_detected ) {
 			return null; // The UI expects this empty response in case of invalid API keys.
 		}
@@ -272,6 +272,9 @@ public static function retrieve( $api ) {
 			// We delete the transient for the account data to trigger the not-connected UI in the admin dashboard.
 			delete_transient( WC_Stripe_Mode::is_test() ? WC_Stripe_Account::TEST_ACCOUNT_OPTION : WC_Stripe_Account::LIVE_ACCOUNT_OPTION );
 
+			// Stripe redacts API keys in the response.
+			WC_Stripe_Logger::log( "Error: GET {$api} returned a 401 " . print_r( $response, true ) );
+
 			return null; // The UI expects this empty response in case of invalid API keys.
 		}