Jetpack_Client - Code Metrics - Inspection of "Update composer.lock after feature branch merge #1..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — update/composer-lock ( 509685 )

unknown

created 2019-06-10 22:58 UTC

Jetpack_Client C

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	384
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

Metric	Value
dl	0
loc	384
rs	5.04
c	0
b	0
f	0
wmc	57
lcom	1
cbo	6

7 Methods

Rating	Name	Size	Complexity
F	remote_request()	128	16
C	_wp_remote_request()	73	14
B	set_time_diff()	27	10
B	wpcom_json_api_request_as_user()	29	7
A	wpcom_json_api_request_as_blog()	27	2
B	_stringify_data()	23	6
A	protocol()	12	2

How to fix Complexity

<?php

class Jetpack_Client {
	const WPCOM_JSON_API_VERSION = '1.1';

	/**
	 * Makes an authorized remote request using Jetpack_Signature
	 *
	 * @return array|WP_Error WP HTTP response on success
	 */
	public static function remote_request( $args, $body = null ) {
		$defaults = array(
			'url' => '',
			'user_id' => 0,
			'blog_id' => 0,
			'auth_location' => JETPACK_CLIENT__AUTH_LOCATION,
			'method' => 'POST',
			'timeout' => 10,
			'redirection' => 0,
			'headers' => array(),
			'stream' => false,
			'filename' => null,
			'sslverify' => true,
		);

		$args = wp_parse_args( $args, $defaults );

		$args['blog_id'] = (int) $args['blog_id'];

		if ( 'header' != $args['auth_location'] ) {
			$args['auth_location'] = 'query_string';
		}

		$token = Jetpack_Data::get_access_token( $args['user_id'] );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		if ( !$token ) {
			return new Jetpack_Error( 'missing_token' );
		}

		$method = strtoupper( $args['method'] );

		$timeout = intval( $args['timeout'] );

		$redirection = $args['redirection'];
		$stream = $args['stream'];
		$filename = $args['filename'];
		$sslverify = $args['sslverify'];

		$request = compact( 'method', 'body', 'timeout', 'redirection', 'stream', 'filename', 'sslverify' );

		@list( $token_key, $secret ) = explode( '.', $token->secret );
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
		if ( empty( $token ) || empty( $secret ) ) {
			return new Jetpack_Error( 'malformed_token' );
		}

		$token_key = sprintf( '%s:%d:%d', $token_key, JETPACK__API_VERSION, $token->external_user_id );

		$time_diff = (int) Jetpack_Options::get_option( 'time_diff' );
		$jetpack_signature = new Jetpack_Signature( $token->secret, $time_diff );

		$timestamp = time() + $time_diff;

		if( function_exists( 'wp_generate_password' ) ) {
			$nonce = wp_generate_password( 10, false );
		} else {
			$nonce = substr( sha1( rand( 0, 1000000 ) ), 0, 10);
		}

		// Kind of annoying.  Maybe refactor Jetpack_Signature to handle body-hashing
		if ( is_null( $body ) ) {
			$body_hash = '';

		} else {
			// Allow arrays to be used in passing data.
			$body_to_hash = $body;

			if ( is_array( $body ) ) {
				// We cast this to a new variable, because the array form of $body needs to be
				// maintained so it can be passed into the request later on in the code.
				if ( count( $body ) > 0 ) {
					$body_to_hash = json_encode( self::_stringify_data( $body ) );
				} else {
					$body_to_hash = '';
				}
			}

			if ( ! is_string( $body_to_hash ) ) {
				return new Jetpack_Error( 'invalid_body', 'Body is malformed.' );
			}

			$body_hash = Jetpack::connection()->sha1_base64( $body_to_hash );
		}

		$auth = array(
			'token' => $token_key,
			'timestamp' => $timestamp,
			'nonce' => $nonce,
			'body-hash' => $body_hash,
		);

		if ( false !== strpos( $args['url'], 'xmlrpc.php' ) ) {
			$url_args = array(
				'for'           => 'jetpack',
				'wpcom_blog_id' => Jetpack_Options::get_option( 'id' ),
			);
		} else {
			$url_args = array();
		}

		if ( 'header' != $args['auth_location'] ) {
			$url_args += $auth;
		}

		$url = add_query_arg( urlencode_deep( $url_args ), $args['url'] );
		$url = Jetpack::fix_url_for_bad_hosts( $url );

		$signature = $jetpack_signature->sign_request( $token_key, $timestamp, $nonce, $body_hash, $method, $url, $body, false );

		if ( !$signature || is_wp_error( $signature ) ) {
			return $signature;
		}

		// Send an Authorization header so various caches/proxies do the right thing
		$auth['signature'] = $signature;
		$auth['version'] = JETPACK__VERSION;
		$header_pieces = array();
		foreach ( $auth as $key => $value ) {
			$header_pieces[] = sprintf( '%s="%s"', $key, $value );
		}
		$request['headers'] = array_merge( $args['headers'], array(
			'Authorization' => "X_JETPACK " . join( ' ', $header_pieces ),
		) );

		if ( 'header' != $args['auth_location'] ) {
			$url = add_query_arg( 'signature', urlencode( $signature ), $url );
		}

		return Jetpack_Client::_wp_remote_request( $url, $request );
	}

	/**
	 * Wrapper for wp_remote_request().  Turns off SSL verification for certain SSL errors.
	 * This is lame, but many, many, many hosts have misconfigured SSL.
	 *
	 * When Jetpack is registered, the jetpack_fallback_no_verify_ssl_certs option is set to the current time if:
	 * 1. a certificate error is found AND
	 * 2. not verifying the certificate works around the problem.
	 *
	 * The option is checked on each request.
	 *
	 * @internal
	 * @see Jetpack::fix_url_for_bad_hosts()
	 *
	 * @return array|WP_Error WP HTTP response on success
	 */
	public static function _wp_remote_request( $url, $args, $set_fallback = false ) {
		/**
		 * SSL verification (`sslverify`) for the JetpackClient remote request
		 * defaults to off, use this filter to force it on.
		 *
		 * Return `true` to ENABLE SSL verification, return `false`
		 * to DISABLE SSL verification.
		 *
		 * @since 3.6.0
		 *
		 * @param bool Whether to force `sslverify` or not.
		 */
		if ( apply_filters( 'jetpack_client_verify_ssl_certs', false ) ) {
			return wp_remote_request( $url, $args );
		}

		$fallback = Jetpack_Options::get_option( 'fallback_no_verify_ssl_certs' );
		if ( false === $fallback ) {
			Jetpack_Options::update_option( 'fallback_no_verify_ssl_certs', 0 );
		}

		if ( (int) $fallback ) {
			// We're flagged to fallback
			$args['sslverify'] = false;
		}

		$response = wp_remote_request( $url, $args );

		if (
			!$set_fallback                                     // We're not allowed to set the flag on this request, so whatever happens happens
		||
			isset( $args['sslverify'] ) && !$args['sslverify'] // No verification - no point in doing it again
		||
			!is_wp_error( $response )                          // Let it ride
		) {
			Jetpack_Client::set_time_diff( $response, $set_fallback );
			return $response;
		}

		// At this point, we're not flagged to fallback and we are allowed to set the flag on this request.

		$message = $response->get_error_message();

		// Is it an SSL Certificate verification error?
		if (
			false === strpos( $message, '14090086' ) // OpenSSL SSL3 certificate error
		&&
			false === strpos( $message, '1407E086' ) // OpenSSL SSL2 certificate error
		&&
			false === strpos( $message, 'error setting certificate verify locations' ) // cURL CA bundle not found
		&&
			false === strpos( $message, 'Peer certificate cannot be authenticated with' ) // cURL CURLE_SSL_CACERT: CA bundle found, but not helpful
			                                                                              // different versions of curl have different error messages
			                                                                              // this string should catch them all
		&&
			false === strpos( $message, 'Problem with the SSL CA cert' ) // cURL CURLE_SSL_CACERT_BADFILE: probably access rights
		) {
			// No, it is not.
			return $response;
		}

		// Redo the request without SSL certificate verification.
		$args['sslverify'] = false;
		$response = wp_remote_request( $url, $args );

		if ( !is_wp_error( $response ) ) {
			// The request went through this time, flag for future fallbacks
			Jetpack_Options::update_option( 'fallback_no_verify_ssl_certs', time() );
			Jetpack_Client::set_time_diff( $response, $set_fallback );
		}

		return $response;
	}

	public static function set_time_diff( &$response, $force_set = false ) {
		$code = wp_remote_retrieve_response_code( $response );

		// Only trust the Date header on some responses
		if ( 200 != $code && 304 != $code && 400 != $code && 401 != $code ) {
			return;
		}

		if ( !$date = wp_remote_retrieve_header( $response, 'date' ) ) {
			return;
		}

		if ( 0 >= $time = (int) strtotime( $date ) ) {
			return;
		}

		$time_diff = $time - time();

		if ( $force_set ) { // during register
			Jetpack_Options::update_option( 'time_diff', $time_diff );
		} else { // otherwise
			$old_diff = Jetpack_Options::get_option( 'time_diff' );
			if ( false === $old_diff || abs( $time_diff - (int) $old_diff ) > 10 ) {
				Jetpack_Options::update_option( 'time_diff', $time_diff );
			}
		}
	}

	/**
	 * Queries the WordPress.com REST API with a user token.
	 *
	 * @param  string $path             REST API path.
	 * @param  string $version          REST API version. Default is `2`.
	 * @param  array  $args             Arguments to {@see WP_Http}. Default is `array()`.
	 * @param  string $body             Body passed to {@see WP_Http}. Default is `null`.

	 * @param  string $base_api_path    REST API root. Default is `wpcom`.
	 *
	 * @return array|WP_Error $response Response data, else {@see WP_Error} on failure.
	 */
	public static function wpcom_json_api_request_as_user( $path, $version = '2', $args = array(), $body = null, $base_api_path = 'wpcom' ) {
		$base_api_path = trim( $base_api_path, '/' );
		$version       = ltrim( $version, 'v' );
		$path          = ltrim( $path, '/' );

		$args = array_intersect_key( $args, array(
			'headers'     => 'array',
			'method'      => 'string',
			'timeout'     => 'int',
			'redirection' => 'int',
			'stream'      => 'boolean',
			'filename'    => 'string',
			'sslverify'   => 'boolean',
		) );

		$args['user_id'] = get_current_user_id();
		$args['method']  = isset( $args['method'] ) ? strtoupper( $args['method'] ) : 'GET';
		$args['url']     = sprintf( '%s://%s/%s/v%s/%s', self::protocol(), JETPACK__WPCOM_JSON_API_HOST, $base_api_path, $version, $path );

		if ( isset( $body ) && ! isset( $args['headers'] ) && in_array( $args['method'], array( 'POST', 'PUT', 'PATCH' ), true ) ) {
			$args['headers'] = array( 'Content-Type' => 'application/json' );
		}

		if ( isset( $body ) && ! is_string( $body ) ) {
			$body = wp_json_encode( $body );
		}

		return self::remote_request( $args, $body );
	}

	/**
	 * Query the WordPress.com REST API using the blog token
	 *
	 * @param string  $path
	 * @param string  $version
	 * @param array   $args
	 * @param string  $body

	 * @param string  $base_api_path
	 * @return array|WP_Error $response Data.
	 */
	static function wpcom_json_api_request_as_blog( $path, $version = self::WPCOM_JSON_API_VERSION, $args = array(), $body = null, $base_api_path = 'rest' ) {
		$filtered_args = array_intersect_key( $args, array(
			'headers'     => 'array',
			'method'      => 'string',
			'timeout'     => 'int',
			'redirection' => 'int',
			'stream'      => 'boolean',
			'filename'    => 'string',
			'sslverify'   => 'boolean',
		) );

		// unprecedingslashit
		$_path = preg_replace( '/^\//', '', $path );

		// Use GET by default whereas `remote_request` uses POST
		$request_method = ( isset( $filtered_args['method'] ) ) ? $filtered_args['method'] : 'GET';

		$url = sprintf( '%s://%s/%s/v%s/%s', self::protocol(), JETPACK__WPCOM_JSON_API_HOST, $base_api_path, $version, $_path );

		$validated_args = array_merge( $filtered_args, array(
			'url'     => $url,
			'blog_id' => (int) Jetpack_Options::get_option( 'id' ),
			'method'  => $request_method,
		) );

		return Jetpack_Client::remote_request( $validated_args, $body );
	}

	/**
	 * Takes an array or similar structure and recursively turns all values into strings. This is used to
	 * make sure that body hashes are made ith the string version, which is what will be seen after a
	 * server pulls up the data in the $_POST array.
	 *
	 * @param array|mixed $data
	 *
	 * @return array|string
	 */
	public static function _stringify_data( $data ) {

		// Booleans are special, lets just makes them and explicit 1/0 instead of the 0 being an empty string.
		if ( is_bool( $data ) ) {
			return $data ? "1" : "0";
		}

		// Cast objects into arrays.
		if ( is_object( $data ) ) {
			$data = (array) $data;
		}

		// Non arrays at this point should be just converted to strings.
		if ( ! is_array( $data ) ) {
			return (string)$data;
		}

		foreach ( $data as $key => &$value ) {
			$value = self::_stringify_data( $value );
		}

		return $data;
	}

	/**
	 * Gets protocol string.
	 *
	 * @return string `https` (if possible), else `http`.
	 */
	public static function protocol() {
		/**
		 * Determines whether Jetpack can send outbound https requests to the WPCOM api.
		 *
		 * @since 3.6.0
		 *
		 * @param bool $proto Defaults to true.
		 */
		$https = apply_filters( 'jetpack_can_make_outbound_https', true );

		return $https ? 'https' : 'http';
	}
}


1			<?php
2
3			class Jetpack_Client {
4			const WPCOM_JSON_API_VERSION = '1.1';
5
6			/**
7			* Makes an authorized remote request using Jetpack_Signature
8			*
9			* @return array\|WP_Error WP HTTP response on success
10			*/
11			public static function remote_request( $args, $body = null ) {
12			$defaults = array(
13			'url' => '',
14			'user_id' => 0,
15			'blog_id' => 0,
16			'auth_location' => JETPACK_CLIENT__AUTH_LOCATION,
17			'method' => 'POST',
18			'timeout' => 10,
19			'redirection' => 0,
20			'headers' => array(),
21			'stream' => false,
22			'filename' => null,
23			'sslverify' => true,
24			);
25
26			$args = wp_parse_args( $args, $defaults );
27
28			$args['blog_id'] = (int) $args['blog_id'];
29
30			if ( 'header' != $args['auth_location'] ) {
31			$args['auth_location'] = 'query_string';
32			}
33
34			$token = Jetpack_Data::get_access_token( $args['user_id'] );
			0 ignored issues – show Documentation introduced 2015-11-20 23:03 UTC by Report Bug Copy Issue Report `$args['user_id']` is of type `integer\|string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Deprecated Code introduced 2019-06-06 00:27 UTC by Report Bug Copy Issue Report The method `Jetpack_Data::get_access_token()` has been deprecated with message: 7.5 Use Connection_Manager instead. This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
35			if ( !$token ) {
36			return new Jetpack_Error( 'missing_token' );
37			}
38
39			$method = strtoupper( $args['method'] );
40
41			$timeout = intval( $args['timeout'] );
42
43			$redirection = $args['redirection'];
44			$stream = $args['stream'];
45			$filename = $args['filename'];
46			$sslverify = $args['sslverify'];
47
48			$request = compact( 'method', 'body', 'timeout', 'redirection', 'stream', 'filename', 'sslverify' );
49
50			@list( $token_key, $secret ) = explode( '.', $token->secret );
			0 ignored issues – show Security Best Practice introduced 2017-04-18 21:07 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
51			if ( empty( $token ) \|\| empty( $secret ) ) {
52			return new Jetpack_Error( 'malformed_token' );
53			}
54
55			$token_key = sprintf( '%s:%d:%d', $token_key, JETPACK__API_VERSION, $token->external_user_id );
56
57			$time_diff = (int) Jetpack_Options::get_option( 'time_diff' );
58			$jetpack_signature = new Jetpack_Signature( $token->secret, $time_diff );
59
60			$timestamp = time() + $time_diff;
61
62			if( function_exists( 'wp_generate_password' ) ) {
63			$nonce = wp_generate_password( 10, false );
64			} else {
65			$nonce = substr( sha1( rand( 0, 1000000 ) ), 0, 10);
66			}
67
68			// Kind of annoying. Maybe refactor Jetpack_Signature to handle body-hashing
69			if ( is_null( $body ) ) {
70			$body_hash = '';
71
72			} else {
73			// Allow arrays to be used in passing data.
74			$body_to_hash = $body;
75
76			if ( is_array( $body ) ) {
77			// We cast this to a new variable, because the array form of $body needs to be
78			// maintained so it can be passed into the request later on in the code.
79			if ( count( $body ) > 0 ) {
80			$body_to_hash = json_encode( self::_stringify_data( $body ) );
81			} else {
82			$body_to_hash = '';
83			}
84			}
85
86			if ( ! is_string( $body_to_hash ) ) {
87			return new Jetpack_Error( 'invalid_body', 'Body is malformed.' );
88			}
89
90			$body_hash = Jetpack::connection()->sha1_base64( $body_to_hash );
91			}
92
93			$auth = array(
94			'token' => $token_key,
95			'timestamp' => $timestamp,
96			'nonce' => $nonce,
97			'body-hash' => $body_hash,
98			);
99
100			if ( false !== strpos( $args['url'], 'xmlrpc.php' ) ) {
101			$url_args = array(
102			'for' => 'jetpack',
103			'wpcom_blog_id' => Jetpack_Options::get_option( 'id' ),
104			);
105			} else {
106			$url_args = array();
107			}
108
109			if ( 'header' != $args['auth_location'] ) {
110			$url_args += $auth;
111			}
112
113			$url = add_query_arg( urlencode_deep( $url_args ), $args['url'] );
114			$url = Jetpack::fix_url_for_bad_hosts( $url );
115
116			$signature = $jetpack_signature->sign_request( $token_key, $timestamp, $nonce, $body_hash, $method, $url, $body, false );
117
118			if ( !$signature \|\| is_wp_error( $signature ) ) {
119			return $signature;
120			}
121
122			// Send an Authorization header so various caches/proxies do the right thing
123			$auth['signature'] = $signature;
124			$auth['version'] = JETPACK__VERSION;
125			$header_pieces = array();
126			foreach ( $auth as $key => $value ) {
127			$header_pieces[] = sprintf( '%s="%s"', $key, $value );
128			}
129			$request['headers'] = array_merge( $args['headers'], array(
130			'Authorization' => "X_JETPACK " . join( ' ', $header_pieces ),
131			) );
132
133			if ( 'header' != $args['auth_location'] ) {
134			$url = add_query_arg( 'signature', urlencode( $signature ), $url );
135			}
136
137			return Jetpack_Client::_wp_remote_request( $url, $request );
138			}
139
140			/**
141			* Wrapper for wp_remote_request(). Turns off SSL verification for certain SSL errors.
142			* This is lame, but many, many, many hosts have misconfigured SSL.
143			*
144			* When Jetpack is registered, the jetpack_fallback_no_verify_ssl_certs option is set to the current time if:
145			* 1. a certificate error is found AND
146			* 2. not verifying the certificate works around the problem.
147			*
148			* The option is checked on each request.
149			*
150			* @internal
151			* @see Jetpack::fix_url_for_bad_hosts()
152			*
153			* @return array\|WP_Error WP HTTP response on success
154			*/
155			public static function _wp_remote_request( $url, $args, $set_fallback = false ) {
156			/**
157			* SSL verification (`sslverify`) for the JetpackClient remote request
158			* defaults to off, use this filter to force it on.
159			*
160			* Return `true` to ENABLE SSL verification, return `false`
161			* to DISABLE SSL verification.
162			*
163			* @since 3.6.0
164			*
165			* @param bool Whether to force `sslverify` or not.
166			*/
167			if ( apply_filters( 'jetpack_client_verify_ssl_certs', false ) ) {
168			return wp_remote_request( $url, $args );
169			}
170
171			$fallback = Jetpack_Options::get_option( 'fallback_no_verify_ssl_certs' );
172			if ( false === $fallback ) {
173			Jetpack_Options::update_option( 'fallback_no_verify_ssl_certs', 0 );
174			}
175
176			if ( (int) $fallback ) {
177			// We're flagged to fallback
178			$args['sslverify'] = false;
179			}
180
181			$response = wp_remote_request( $url, $args );
182
183			if (
184			!$set_fallback // We're not allowed to set the flag on this request, so whatever happens happens
185			\|\|
186			isset( $args['sslverify'] ) && !$args['sslverify'] // No verification - no point in doing it again
187			\|\|
188			!is_wp_error( $response ) // Let it ride
189			) {
190			Jetpack_Client::set_time_diff( $response, $set_fallback );
191			return $response;
192			}
193
194			// At this point, we're not flagged to fallback and we are allowed to set the flag on this request.
195
196			$message = $response->get_error_message();
197
198			// Is it an SSL Certificate verification error?
199			if (
200			false === strpos( $message, '14090086' ) // OpenSSL SSL3 certificate error
201			&&
202			false === strpos( $message, '1407E086' ) // OpenSSL SSL2 certificate error
203			&&
204			false === strpos( $message, 'error setting certificate verify locations' ) // cURL CA bundle not found
205			&&
206			false === strpos( $message, 'Peer certificate cannot be authenticated with' ) // cURL CURLE_SSL_CACERT: CA bundle found, but not helpful
207			// different versions of curl have different error messages
208			// this string should catch them all
209			&&
210			false === strpos( $message, 'Problem with the SSL CA cert' ) // cURL CURLE_SSL_CACERT_BADFILE: probably access rights
211			) {
212			// No, it is not.
213			return $response;
214			}
215
216			// Redo the request without SSL certificate verification.
217			$args['sslverify'] = false;
218			$response = wp_remote_request( $url, $args );
219
220			if ( !is_wp_error( $response ) ) {
221			// The request went through this time, flag for future fallbacks
222			Jetpack_Options::update_option( 'fallback_no_verify_ssl_certs', time() );
223			Jetpack_Client::set_time_diff( $response, $set_fallback );
224			}
225
226			return $response;
227			}
228
229			public static function set_time_diff( &$response, $force_set = false ) {
230			$code = wp_remote_retrieve_response_code( $response );
231
232			// Only trust the Date header on some responses
233			if ( 200 != $code && 304 != $code && 400 != $code && 401 != $code ) {
234			return;
235			}
236
237			if ( !$date = wp_remote_retrieve_header( $response, 'date' ) ) {
238			return;
239			}
240
241			if ( 0 >= $time = (int) strtotime( $date ) ) {
242			return;
243			}
244
245			$time_diff = $time - time();
246
247			if ( $force_set ) { // during register
248			Jetpack_Options::update_option( 'time_diff', $time_diff );
249			} else { // otherwise
250			$old_diff = Jetpack_Options::get_option( 'time_diff' );
251			if ( false === $old_diff \|\| abs( $time_diff - (int) $old_diff ) > 10 ) {
252			Jetpack_Options::update_option( 'time_diff', $time_diff );
253			}
254			}
255			}
256
257			/**
258			* Queries the WordPress.com REST API with a user token.
259			*
260			* @param string $path REST API path.
261			* @param string $version REST API version. Default is `2`.
262			* @param array $args Arguments to {@see WP_Http}. Default is `array()`.
263			* @param string $body Body passed to {@see WP_Http}. Default is `null`.
			0 ignored issues – show Documentation introduced 2018-03-22 15:12 UTC by Report Bug Copy Issue Report Should the type for parameter `$body` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
264			* @param string $base_api_path REST API root. Default is `wpcom`.
265			*
266			* @return array\|WP_Error $response Response data, else {@see WP_Error} on failure.
267			*/
268			public static function wpcom_json_api_request_as_user( $path, $version = '2', $args = array(), $body = null, $base_api_path = 'wpcom' ) {
269			$base_api_path = trim( $base_api_path, '/' );
270			$version = ltrim( $version, 'v' );
271			$path = ltrim( $path, '/' );
272
273			$args = array_intersect_key( $args, array(
274			'headers' => 'array',
275			'method' => 'string',
276			'timeout' => 'int',
277			'redirection' => 'int',
278			'stream' => 'boolean',
279			'filename' => 'string',
280			'sslverify' => 'boolean',
281			) );
282
283			$args['user_id'] = get_current_user_id();
284			$args['method'] = isset( $args['method'] ) ? strtoupper( $args['method'] ) : 'GET';
285			$args['url'] = sprintf( '%s://%s/%s/v%s/%s', self::protocol(), JETPACK__WPCOM_JSON_API_HOST, $base_api_path, $version, $path );
286
287			if ( isset( $body ) && ! isset( $args['headers'] ) && in_array( $args['method'], array( 'POST', 'PUT', 'PATCH' ), true ) ) {
288			$args['headers'] = array( 'Content-Type' => 'application/json' );
289			}
290
291			if ( isset( $body ) && ! is_string( $body ) ) {
292			$body = wp_json_encode( $body );
293			}
294
295			return self::remote_request( $args, $body );
296			}
297
298			/**
299			* Query the WordPress.com REST API using the blog token
300			*
301			* @param string $path
302			* @param string $version
303			* @param array $args
304			* @param string $body
			0 ignored issues – show Documentation introduced 2018-02-13 15:03 UTC by Report Bug Copy Issue Report Should the type for parameter `$body` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
305			* @param string $base_api_path
306			* @return array\|WP_Error $response Data.
307			*/
308			static function wpcom_json_api_request_as_blog( $path, $version = self::WPCOM_JSON_API_VERSION, $args = array(), $body = null, $base_api_path = 'rest' ) {
309			$filtered_args = array_intersect_key( $args, array(
310			'headers' => 'array',
311			'method' => 'string',
312			'timeout' => 'int',
313			'redirection' => 'int',
314			'stream' => 'boolean',
315			'filename' => 'string',
316			'sslverify' => 'boolean',
317			) );
318
319			// unprecedingslashit
320			$_path = preg_replace( '/^\//', '', $path );
321
322			// Use GET by default whereas `remote_request` uses POST
323			$request_method = ( isset( $filtered_args['method'] ) ) ? $filtered_args['method'] : 'GET';
324
325			$url = sprintf( '%s://%s/%s/v%s/%s', self::protocol(), JETPACK__WPCOM_JSON_API_HOST, $base_api_path, $version, $_path );
326
327			$validated_args = array_merge( $filtered_args, array(
328			'url' => $url,
329			'blog_id' => (int) Jetpack_Options::get_option( 'id' ),
330			'method' => $request_method,
331			) );
332
333			return Jetpack_Client::remote_request( $validated_args, $body );
334			}
335
336			/**
337			* Takes an array or similar structure and recursively turns all values into strings. This is used to
338			* make sure that body hashes are made ith the string version, which is what will be seen after a
339			* server pulls up the data in the $_POST array.
340			*
341			* @param array\|mixed $data
342			*
343			* @return array\|string
344			*/
345			public static function _stringify_data( $data ) {
346
347			// Booleans are special, lets just makes them and explicit 1/0 instead of the 0 being an empty string.
348			if ( is_bool( $data ) ) {
349			return $data ? "1" : "0";
350			}
351
352			// Cast objects into arrays.
353			if ( is_object( $data ) ) {
354			$data = (array) $data;
355			}
356
357			// Non arrays at this point should be just converted to strings.
358			if ( ! is_array( $data ) ) {
359			return (string)$data;
360			}
361
362			foreach ( $data as $key => &$value ) {
363			$value = self::_stringify_data( $value );
364			}
365
366			return $data;
367			}
368
369			/**
370			* Gets protocol string.
371			*
372			* @return string `https` (if possible), else `http`.
373			*/
374			public static function protocol() {
375			/**
376			* Determines whether Jetpack can send outbound https requests to the WPCOM api.
377			*
378			* @since 3.6.0
379			*
380			* @param bool $proto Defaults to true.
381			*/
382			$https = apply_filters( 'jetpack_can_make_outbound_https', true );
383
384			return $https ? 'https' : 'http';
385			}
386			}
387

Automattic / jetpack

Push — update/composer-lock ( 509685 )

Jetpack_Client C

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like