Completed
Push — add/single-product-backup-body... ( 4e109f...285f8e )
by
unknown
16:57 queued 09:31
created

Manager::authorize()   C

Complexity

Conditions 12
Paths 12

Size

Total Lines 80

Duplication

Lines 7
Ratio 8.75 %

Importance

Changes 0
Metric Value
cc 12
nc 12
nop 1
dl 7
loc 80
rs 6.0096
c 0
b 0
f 0

How to fix   Long Method    Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
/**
3
 * The Jetpack Connection manager class file.
4
 *
5
 * @package automattic/jetpack-connection
6
 */
7
8
namespace Automattic\Jetpack\Connection;
9
10
use Automattic\Jetpack\Constants;
11
use Automattic\Jetpack\Roles;
12
use Automattic\Jetpack\Tracking;
13
14
/**
15
 * The Jetpack Connection Manager class that is used as a single gateway between WordPress.com
16
 * and Jetpack.
17
 */
18
class Manager {
19
20
	const SECRETS_MISSING        = 'secrets_missing';
21
	const SECRETS_EXPIRED        = 'secrets_expired';
22
	const SECRETS_OPTION_NAME    = 'jetpack_secrets';
23
	const MAGIC_NORMAL_TOKEN_KEY = ';normal;';
24
	const JETPACK_MASTER_USER    = true;
25
26
	/**
27
	 * The procedure that should be run to generate secrets.
28
	 *
29
	 * @var Callable
30
	 */
31
	protected $secret_callable;
32
33
	/**
34
	 * A copy of the raw POST data for signature verification purposes.
35
	 *
36
	 * @var String
37
	 */
38
	protected $raw_post_data;
39
40
	/**
41
	 * Verification data needs to be stored to properly verify everything.
42
	 *
43
	 * @var Object
44
	 */
45
	private $xmlrpc_verification = null;
46
47
	/**
48
	 * Initializes required listeners. This is done separately from the constructors
49
	 * because some objects sometimes need to instantiate separate objects of this class.
50
	 *
51
	 * @todo Implement a proper nonce verification.
52
	 */
53
	public function init() {
54
		$this->setup_xmlrpc_handlers(
55
			$_GET, // phpcs:ignore WordPress.Security.NonceVerification.Recommended
56
			$this->is_active(),
57
			$this->verify_xml_rpc_signature()
0 ignored issues
show
Bug introduced by
It seems like $this->verify_xml_rpc_signature() targeting Automattic\Jetpack\Conne...ify_xml_rpc_signature() can also be of type array; however, Automattic\Jetpack\Conne...setup_xmlrpc_handlers() does only seem to accept boolean, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
58
		);
59
60
		if ( $this->is_active() ) {
61
			add_filter( 'xmlrpc_methods', array( $this, 'public_xmlrpc_methods' ) );
62
		} else {
63
			add_action( 'rest_api_init', array( $this, 'initialize_rest_api_registration_connector' ) );
64
		}
65
66
		add_action( 'jetpack_clean_nonces', array( $this, 'clean_nonces' ) );
67
		if ( ! wp_next_scheduled( 'jetpack_clean_nonces' ) ) {
68
			wp_schedule_event( time(), 'hourly', 'jetpack_clean_nonces' );
69
		}
70
	}
71
72
	/**
73
	 * Sets up the XMLRPC request handlers.
74
	 *
75
	 * @param Array                  $request_params incoming request parameters.
76
	 * @param Boolean                $is_active whether the connection is currently active.
77
	 * @param Boolean                $is_signed whether the signature check has been successful.
78
	 * @param \Jetpack_XMLRPC_Server $xmlrpc_server (optional) an instance of the server to use instead of instantiating a new one.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $xmlrpc_server not be null|\Jetpack_XMLRPC_Server?

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...
79
	 */
80
	public function setup_xmlrpc_handlers(
81
		$request_params,
82
		$is_active,
83
		$is_signed,
84
		\Jetpack_XMLRPC_Server $xmlrpc_server = null
85
	) {
86
		add_filter( 'xmlrpc_blog_options', array( $this, 'xmlrpc_options' ), 1000, 2 );
87
88
		if (
89
			! isset( $request_params['for'] )
90
			|| 'jetpack' !== $request_params['for']
91
		) {
92
			return false;
93
		}
94
95
		// Alternate XML-RPC, via ?for=jetpack&jetpack=comms.
96
		if (
97
			isset( $request_params['jetpack'] )
98
			&& 'comms' === $request_params['jetpack']
99
		) {
100
			if ( ! Constants::is_defined( 'XMLRPC_REQUEST' ) ) {
101
				// Use the real constant here for WordPress' sake.
102
				define( 'XMLRPC_REQUEST', true );
103
			}
104
105
			add_action( 'template_redirect', array( $this, 'alternate_xmlrpc' ) );
106
107
			add_filter( 'xmlrpc_methods', array( $this, 'remove_non_jetpack_xmlrpc_methods' ), 1000 );
108
		}
109
110
		if ( ! Constants::get_constant( 'XMLRPC_REQUEST' ) ) {
111
			return false;
112
		}
113
		// Display errors can cause the XML to be not well formed.
114
		@ini_set( 'display_errors', false ); // phpcs:ignore
0 ignored issues
show
Security Best Practice introduced by
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...
115
116
		if ( $xmlrpc_server ) {
117
			$this->xmlrpc_server = $xmlrpc_server;
0 ignored issues
show
Bug introduced by
The property xmlrpc_server does not exist. Did you maybe forget to declare it?

In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code:

class MyClass { }

$x = new MyClass();
$x->foo = true;

Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion:

class MyClass {
    public $foo;
}

$x = new MyClass();
$x->foo = true;
Loading history...
118
		} else {
119
			$this->xmlrpc_server = new \Jetpack_XMLRPC_Server();
120
		}
121
122
		$this->require_jetpack_authentication();
123
124
		if ( $is_active ) {
125
			// Hack to preserve $HTTP_RAW_POST_DATA.
126
			add_filter( 'xmlrpc_methods', array( $this, 'xmlrpc_methods' ) );
127
128
			if ( $is_signed ) {
129
				// The actual API methods.
130
				add_filter( 'xmlrpc_methods', array( $this->xmlrpc_server, 'xmlrpc_methods' ) );
131
			} else {
132
				// The jetpack.authorize method should be available for unauthenticated users on a site with an
133
				// active Jetpack connection, so that additional users can link their account.
134
				add_filter( 'xmlrpc_methods', array( $this->xmlrpc_server, 'authorize_xmlrpc_methods' ) );
135
			}
136
		} else {
137
			// The bootstrap API methods.
138
			add_filter( 'xmlrpc_methods', array( $this->xmlrpc_server, 'bootstrap_xmlrpc_methods' ) );
139
140
			if ( $is_signed ) {
141
				// The jetpack Provision method is available for blog-token-signed requests.
142
				add_filter( 'xmlrpc_methods', array( $this->xmlrpc_server, 'provision_xmlrpc_methods' ) );
143
			} else {
144
				new XMLRPC_Connector( $this );
145
			}
146
		}
147
148
		// Now that no one can authenticate, and we're whitelisting all XML-RPC methods, force enable_xmlrpc on.
149
		add_filter( 'pre_option_enable_xmlrpc', '__return_true' );
150
		return true;
151
	}
152
153
	/**
154
	 * Initializes the REST API connector on the init hook.
155
	 */
156
	public function initialize_rest_api_registration_connector() {
157
		new REST_Connector( $this );
158
	}
159
160
	/**
161
	 * Since a lot of hosts use a hammer approach to "protecting" WordPress sites,
162
	 * and just blanket block all requests to /xmlrpc.php, or apply other overly-sensitive
163
	 * security/firewall policies, we provide our own alternate XML RPC API endpoint
164
	 * which is accessible via a different URI. Most of the below is copied directly
165
	 * from /xmlrpc.php so that we're replicating it as closely as possible.
166
	 *
167
	 * @todo Tighten $wp_xmlrpc_server_class a bit to make sure it doesn't do bad things.
168
	 */
169
	public function alternate_xmlrpc() {
170
		// phpcs:disable PHPCompatibility.Variables.RemovedPredefinedGlobalVariables.http_raw_post_dataDeprecatedRemoved
171
		// phpcs:disable WordPress.WP.GlobalVariablesOverride.Prohibited
172
		global $HTTP_RAW_POST_DATA;
173
174
		// Some browser-embedded clients send cookies. We don't want them.
175
		$_COOKIE = array();
176
177
		// A fix for mozBlog and other cases where '<?xml' isn't on the very first line.
178
		if ( isset( $HTTP_RAW_POST_DATA ) ) {
179
			$HTTP_RAW_POST_DATA = trim( $HTTP_RAW_POST_DATA );
180
		}
181
182
		// phpcs:enable
183
184
		include_once ABSPATH . 'wp-admin/includes/admin.php';
185
		include_once ABSPATH . WPINC . '/class-IXR.php';
186
		include_once ABSPATH . WPINC . '/class-wp-xmlrpc-server.php';
187
188
		/**
189
		 * Filters the class used for handling XML-RPC requests.
190
		 *
191
		 * @since 3.1.0
192
		 *
193
		 * @param string $class The name of the XML-RPC server class.
194
		 */
195
		$wp_xmlrpc_server_class = apply_filters( 'wp_xmlrpc_server_class', 'wp_xmlrpc_server' );
196
		$wp_xmlrpc_server       = new $wp_xmlrpc_server_class();
197
198
		// Fire off the request.
199
		nocache_headers();
200
		$wp_xmlrpc_server->serve_request();
201
202
		exit;
203
	}
204
205
	/**
206
	 * Removes all XML-RPC methods that are not `jetpack.*`.
207
	 * Only used in our alternate XML-RPC endpoint, where we want to
208
	 * ensure that Core and other plugins' methods are not exposed.
209
	 *
210
	 * @param array $methods a list of registered WordPress XMLRPC methods.
211
	 * @return array filtered $methods
212
	 */
213
	public function remove_non_jetpack_xmlrpc_methods( $methods ) {
214
		$jetpack_methods = array();
215
216
		foreach ( $methods as $method => $callback ) {
217
			if ( 0 === strpos( $method, 'jetpack.' ) ) {
218
				$jetpack_methods[ $method ] = $callback;
219
			}
220
		}
221
222
		return $jetpack_methods;
223
	}
224
225
	/**
226
	 * Removes all other authentication methods not to allow other
227
	 * methods to validate unauthenticated requests.
228
	 */
229
	public function require_jetpack_authentication() {
230
		// Don't let anyone authenticate.
231
		$_COOKIE = array();
232
		remove_all_filters( 'authenticate' );
233
		remove_all_actions( 'wp_login_failed' );
234
235
		if ( $this->is_active() ) {
236
			// Allow Jetpack authentication.
237
			add_filter( 'authenticate', array( $this, 'authenticate_jetpack' ), 10, 3 );
238
		}
239
	}
240
241
	/**
242
	 * Authenticates XML-RPC and other requests from the Jetpack Server
243
	 *
244
	 * @param WP_User|Mixed $user user object if authenticated.
245
	 * @param String        $username username.
246
	 * @param String        $password password string.
247
	 * @return WP_User|Mixed authenticated user or error.
248
	 */
249
	public function authenticate_jetpack( $user, $username, $password ) {
250
		if ( is_a( $user, '\\WP_User' ) ) {
251
			return $user;
252
		}
253
254
		$token_details = $this->verify_xml_rpc_signature();
255
256
		if ( ! $token_details ) {
257
			return $user;
258
		}
259
260
		if ( 'user' !== $token_details['type'] ) {
261
			return $user;
262
		}
263
264
		if ( ! $token_details['user_id'] ) {
265
			return $user;
266
		}
267
268
		nocache_headers();
269
270
		return new \WP_User( $token_details['user_id'] );
271
	}
272
273
	/**
274
	 * Verifies the signature of the current request.
275
	 *
276
	 * @return false|array
277
	 */
278
	public function verify_xml_rpc_signature() {
279
		if ( is_null( $this->xmlrpc_verification ) ) {
280
			$this->xmlrpc_verification = $this->internal_verify_xml_rpc_signature();
281
282
			if ( is_wp_error( $this->xmlrpc_verification ) ) {
283
				/**
284
				 * Action for logging XMLRPC signature verification errors. This data is sensitive.
285
				 *
286
				 * Error codes:
287
				 * - malformed_token
288
				 * - malformed_user_id
289
				 * - unknown_token
290
				 * - could_not_sign
291
				 * - invalid_nonce
292
				 * - signature_mismatch
293
				 *
294
				 * @since 7.5.0
295
				 *
296
				 * @param WP_Error $signature_verification_error The verification error
297
				 */
298
				do_action( 'jetpack_verify_signature_error', $this->xmlrpc_verification );
299
			}
300
		}
301
302
		return is_wp_error( $this->xmlrpc_verification ) ? false : $this->xmlrpc_verification;
303
	}
304
305
	/**
306
	 * Verifies the signature of the current request.
307
	 *
308
	 * This function has side effects and should not be used. Instead,
309
	 * use the memoized version `->verify_xml_rpc_signature()`.
310
	 *
311
	 * @internal
312
	 * @todo Refactor to use proper nonce verification.
313
	 */
314
	private function internal_verify_xml_rpc_signature() {
315
		// phpcs:disable WordPress.Security.NonceVerification.Recommended
316
		// It's not for us.
317
		if ( ! isset( $_GET['token'] ) || empty( $_GET['signature'] ) ) {
318
			return false;
319
		}
320
321
		$signature_details = array(
322
			'token'     => isset( $_GET['token'] ) ? wp_unslash( $_GET['token'] ) : '',
323
			'timestamp' => isset( $_GET['timestamp'] ) ? wp_unslash( $_GET['timestamp'] ) : '',
324
			'nonce'     => isset( $_GET['nonce'] ) ? wp_unslash( $_GET['nonce'] ) : '',
325
			'body_hash' => isset( $_GET['body-hash'] ) ? wp_unslash( $_GET['body-hash'] ) : '',
326
			'method'    => wp_unslash( $_SERVER['REQUEST_METHOD'] ),
327
			'url'       => wp_unslash( $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'] ), // Temp - will get real signature URL later.
328
			'signature' => isset( $_GET['signature'] ) ? wp_unslash( $_GET['signature'] ) : '',
329
		);
330
331
		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
332
		@list( $token_key, $version, $user_id ) = explode( ':', wp_unslash( $_GET['token'] ) );
0 ignored issues
show
Security Best Practice introduced by
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...
333
		// phpcs:enable WordPress.Security.NonceVerification.Recommended
334
335
		if (
336
			empty( $token_key )
337
		||
338
			empty( $version ) || strval( JETPACK__API_VERSION ) !== $version
339
		) {
340
			return new \WP_Error( 'malformed_token', 'Malformed token in request', compact( 'signature_details' ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'malformed_token'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
341
		}
342
343
		if ( '0' === $user_id ) {
344
			$token_type = 'blog';
345
			$user_id    = 0;
346
		} else {
347
			$token_type = 'user';
348
			if ( empty( $user_id ) || ! ctype_digit( $user_id ) ) {
349
				return new \WP_Error(
350
					'malformed_user_id',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'malformed_user_id'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
351
					'Malformed user_id in request',
352
					compact( 'signature_details' )
353
				);
354
			}
355
			$user_id = (int) $user_id;
356
357
			$user = new \WP_User( $user_id );
358
			if ( ! $user || ! $user->exists() ) {
359
				return new \WP_Error(
360
					'unknown_user',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'unknown_user'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
361
					sprintf( 'User %d does not exist', $user_id ),
362
					compact( 'signature_details' )
363
				);
364
			}
365
		}
366
367
		$token = $this->get_access_token( $user_id, $token_key, false );
368
		if ( is_wp_error( $token ) ) {
369
			$token->add_data( compact( 'signature_details' ) );
370
			return $token;
371
		} elseif ( ! $token ) {
372
			return new \WP_Error(
373
				'unknown_token',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'unknown_token'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
374
				sprintf( 'Token %s:%s:%d does not exist', $token_key, $version, $user_id ),
375
				compact( 'signature_details' )
376
			);
377
		}
378
379
		$jetpack_signature = new \Jetpack_Signature( $token->secret, (int) \Jetpack_Options::get_option( 'time_diff' ) );
380
		// phpcs:disable WordPress.Security.NonceVerification.Missing
381
		if ( isset( $_POST['_jetpack_is_multipart'] ) ) {
382
			$post_data   = $_POST;
383
			$file_hashes = array();
384
			foreach ( $post_data as $post_data_key => $post_data_value ) {
385
				if ( 0 !== strpos( $post_data_key, '_jetpack_file_hmac_' ) ) {
386
					continue;
387
				}
388
				$post_data_key                 = substr( $post_data_key, strlen( '_jetpack_file_hmac_' ) );
389
				$file_hashes[ $post_data_key ] = $post_data_value;
390
			}
391
392
			foreach ( $file_hashes as $post_data_key => $post_data_value ) {
393
				unset( $post_data[ "_jetpack_file_hmac_{$post_data_key}" ] );
394
				$post_data[ $post_data_key ] = $post_data_value;
395
			}
396
397
			ksort( $post_data );
398
399
			$body = http_build_query( stripslashes_deep( $post_data ) );
400
		} elseif ( is_null( $this->raw_post_data ) ) {
401
			$body = file_get_contents( 'php://input' );
402
		} else {
403
			$body = null;
404
		}
405
		// phpcs:enable
406
407
		$signature = $jetpack_signature->sign_current_request(
408
			array( 'body' => is_null( $body ) ? $this->raw_post_data : $body )
409
		);
410
411
		$signature_details['url'] = $jetpack_signature->current_request_url;
412
413
		if ( ! $signature ) {
414
			return new \WP_Error(
415
				'could_not_sign',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'could_not_sign'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
416
				'Unknown signature error',
417
				compact( 'signature_details' )
418
			);
419
		} elseif ( is_wp_error( $signature ) ) {
420
			return $signature;
421
		}
422
423
		// phpcs:disable WordPress.Security.NonceVerification.Recommended
424
		$timestamp = (int) $_GET['timestamp'];
425
		$nonce     = stripslashes( (string) $_GET['nonce'] );
426
		// phpcs:enable WordPress.Security.NonceVerification.Recommended
427
428
		// Use up the nonce regardless of whether the signature matches.
429
		if ( ! $this->add_nonce( $timestamp, $nonce ) ) {
430
			return new \WP_Error(
431
				'invalid_nonce',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'invalid_nonce'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
432
				'Could not add nonce',
433
				compact( 'signature_details' )
434
			);
435
		}
436
437
		// Be careful about what you do with this debugging data.
438
		// If a malicious requester has access to the expected signature,
439
		// bad things might be possible.
440
		$signature_details['expected'] = $signature;
441
442
		// phpcs:ignore WordPress.Security.NonceVerification.Recommended
443
		if ( ! hash_equals( $signature, $_GET['signature'] ) ) {
444
			return new \WP_Error(
445
				'signature_mismatch',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'signature_mismatch'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
446
				'Signature mismatch',
447
				compact( 'signature_details' )
448
			);
449
		}
450
451
		/**
452
		 * Action for additional token checking.
453
		 *
454
		 * @since 7.7.0
455
		 *
456
		 * @param Array $post_data request data.
457
		 * @param Array $token_data token data.
458
		 */
459
		return apply_filters(
460
			'jetpack_signature_check_token',
461
			array(
462
				'type'      => $token_type,
463
				'token_key' => $token_key,
464
				'user_id'   => $token->external_user_id,
465
			),
466
			$token,
467
			$this->raw_post_data
468
		);
469
	}
470
471
	/**
472
	 * Returns true if the current site is connected to WordPress.com.
473
	 *
474
	 * @return Boolean is the site connected?
475
	 */
476
	public function is_active() {
477
		return (bool) $this->get_access_token( self::JETPACK_MASTER_USER );
0 ignored issues
show
Documentation introduced by
self::JETPACK_MASTER_USER is of type boolean, but the function expects a false|integer.

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...
478
	}
479
480
	/**
481
	 * Returns true if the site has both a token and a blog id, which indicates a site has been registered.
482
	 *
483
	 * @access public
484
	 *
485
	 * @return bool
486
	 */
487
	public function is_registered() {
488
		$blog_id   = \Jetpack_Options::get_option( 'id' );
489
		$has_token = $this->is_active();
490
		return $blog_id && $has_token;
491
	}
492
493
	/**
494
	 * Checks to see if the connection owner of the site is missing.
495
	 *
496
	 * @return bool
497
	 */
498
	public function is_missing_connection_owner() {
499
		$connection_owner = $this->get_connection_owner_id();
500
		if ( ! get_user_by( 'id', $connection_owner ) ) {
501
			return true;
502
		}
503
504
		return false;
505
	}
506
507
	/**
508
	 * Returns true if the user with the specified identifier is connected to
509
	 * WordPress.com.
510
	 *
511
	 * @param Integer|Boolean $user_id the user identifier.
512
	 * @return Boolean is the user connected?
513
	 */
514
	public function is_user_connected( $user_id = false ) {
515
		$user_id = false === $user_id ? get_current_user_id() : absint( $user_id );
516
		if ( ! $user_id ) {
517
			return false;
518
		}
519
520
		return (bool) $this->get_access_token( $user_id );
521
	}
522
523
	/**
524
	 * Returns the local user ID of the connection owner.
525
	 *
526
	 * @return string|int Returns the ID of the connection owner or False if no connection owner found.
527
	 */
528 View Code Duplication
	public function get_connection_owner_id() {
529
		$user_token       = $this->get_access_token( JETPACK_MASTER_USER );
0 ignored issues
show
Documentation introduced by
JETPACK_MASTER_USER is of type boolean, but the function expects a false|integer.

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...
530
		$connection_owner = false;
531
		if ( $user_token && is_object( $user_token ) && isset( $user_token->external_user_id ) ) {
532
			$connection_owner = $user_token->external_user_id;
533
		}
534
535
		return $connection_owner;
536
	}
537
538
	/**
539
	 * Returns an array of user_id's that have user tokens for communicating with wpcom.
540
	 * Able to select by specific capability.
541
	 *
542
	 * @param string $capability The capability of the user.
543
	 * @return array Array of WP_User objects if found.
544
	 */
545
	public function get_connected_users( $capability = 'any' ) {
546
		$connected_users    = array();
547
		$connected_user_ids = array_keys( \Jetpack_Options::get_option( 'user_tokens' ) );
548
549
		if ( ! empty( $connected_user_ids ) ) {
550
			foreach ( $connected_user_ids as $id ) {
551
				// Check for capability.
552
				if ( 'any' !== $capability && ! user_can( $id, $capability ) ) {
553
					continue;
554
				}
555
556
				$connected_users[] = get_userdata( $id );
557
			}
558
		}
559
560
		return $connected_users;
561
	}
562
563
	/**
564
	 * Get the wpcom user data of the current|specified connected user.
565
	 *
566
	 * @todo Refactor to properly load the XMLRPC client independently.
567
	 *
568
	 * @param Integer $user_id the user identifier.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user_id not be integer|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...
569
	 * @return Object the user object.
570
	 */
571 View Code Duplication
	public function get_connected_user_data( $user_id = null ) {
572
		if ( ! $user_id ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $user_id of type integer|null is loosely compared to false; this is ambiguous if the integer can be zero. You might want to explicitly use === null instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For integer values, zero is a special case, in particular the following results might be unexpected:

0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
Loading history...
573
			$user_id = get_current_user_id();
574
		}
575
576
		$transient_key    = "jetpack_connected_user_data_$user_id";
577
		$cached_user_data = get_transient( $transient_key );
578
579
		if ( $cached_user_data ) {
580
			return $cached_user_data;
581
		}
582
583
		$xml = new \Jetpack_IXR_Client(
584
			array(
585
				'user_id' => $user_id,
586
			)
587
		);
588
		$xml->query( 'wpcom.getUser' );
589
		if ( ! $xml->isError() ) {
590
			$user_data = $xml->getResponse();
591
			set_transient( $transient_key, $xml->getResponse(), DAY_IN_SECONDS );
592
			return $user_data;
593
		}
594
595
		return false;
596
	}
597
598
	/**
599
	 * Returns a user object of the connection owner.
600
	 *
601
	 * @return object|false False if no connection owner found.
602
	 */
603 View Code Duplication
	public function get_connection_owner() {
604
		$user_token = $this->get_access_token( JETPACK_MASTER_USER );
0 ignored issues
show
Documentation introduced by
JETPACK_MASTER_USER is of type boolean, but the function expects a false|integer.

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...
605
606
		$connection_owner = false;
607
		if ( $user_token && is_object( $user_token ) && isset( $user_token->external_user_id ) ) {
608
			$connection_owner = get_userdata( $user_token->external_user_id );
609
		}
610
611
		return $connection_owner;
612
	}
613
614
	/**
615
	 * Returns true if the provided user is the Jetpack connection owner.
616
	 * If user ID is not specified, the current user will be used.
617
	 *
618
	 * @param Integer|Boolean $user_id the user identifier. False for current user.
619
	 * @return Boolean True the user the connection owner, false otherwise.
620
	 */
621 View Code Duplication
	public function is_connection_owner( $user_id = false ) {
622
		if ( ! $user_id ) {
623
			$user_id = get_current_user_id();
624
		}
625
626
		$user_token = $this->get_access_token( JETPACK_MASTER_USER );
0 ignored issues
show
Documentation introduced by
JETPACK_MASTER_USER is of type boolean, but the function expects a false|integer.

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...
627
628
		return $user_token && is_object( $user_token ) && isset( $user_token->external_user_id ) && $user_id === $user_token->external_user_id;
629
	}
630
631
	/**
632
	 * Connects the user with a specified ID to a WordPress.com user using the
633
	 * remote login flow.
634
	 *
635
	 * @access public
636
	 *
637
	 * @param Integer $user_id (optional) the user identifier, defaults to current user.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user_id not be integer|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...
638
	 * @param String  $redirect_url the URL to redirect the user to for processing, defaults to
0 ignored issues
show
Documentation introduced by
Should the type for parameter $redirect_url 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...
639
	 *                              admin_url().
640
	 * @return WP_Error only in case of a failed user lookup.
641
	 */
642
	public function connect_user( $user_id = null, $redirect_url = null ) {
643
		$user = null;
0 ignored issues
show
Unused Code introduced by
$user is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
644
		if ( null === $user_id ) {
645
			$user = wp_get_current_user();
646
		} else {
647
			$user = get_user_by( 'ID', $user_id );
648
		}
649
650
		if ( empty( $user ) ) {
651
			return new \WP_Error( 'user_not_found', 'Attempting to connect a non-existent user.' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'user_not_found'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
652
		}
653
654
		if ( null === $redirect_url ) {
655
			$redirect_url = admin_url();
0 ignored issues
show
Unused Code introduced by
$redirect_url is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
656
		}
657
658
		// Using wp_redirect intentionally because we're redirecting outside.
659
		wp_redirect( $this->get_authorization_url( $user ) ); // phpcs:ignore WordPress.Security.SafeRedirect
660
		exit();
661
	}
662
663
	/**
664
	 * Unlinks the current user from the linked WordPress.com user.
665
	 *
666
	 * @access public
667
	 * @static
668
	 *
669
	 * @todo Refactor to properly load the XMLRPC client independently.
670
	 *
671
	 * @param Integer $user_id the user identifier.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user_id not be integer|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...
672
	 * @return Boolean Whether the disconnection of the user was successful.
673
	 */
674
	public static function disconnect_user( $user_id = null ) {
675
		$tokens = \Jetpack_Options::get_option( 'user_tokens' );
676
		if ( ! $tokens ) {
677
			return false;
678
		}
679
680
		$user_id = empty( $user_id ) ? get_current_user_id() : intval( $user_id );
681
682
		if ( \Jetpack_Options::get_option( 'master_user' ) === $user_id ) {
683
			return false;
684
		}
685
686
		if ( ! isset( $tokens[ $user_id ] ) ) {
687
			return false;
688
		}
689
690
		$xml = new \Jetpack_IXR_Client( compact( 'user_id' ) );
691
		$xml->query( 'jetpack.unlink_user', $user_id );
692
693
		unset( $tokens[ $user_id ] );
694
695
		\Jetpack_Options::update_option( 'user_tokens', $tokens );
696
697
		/**
698
		 * Fires after the current user has been unlinked from WordPress.com.
699
		 *
700
		 * @since 4.1.0
701
		 *
702
		 * @param int $user_id The current user's ID.
703
		 */
704
		do_action( 'jetpack_unlinked_user', $user_id );
705
706
		return true;
707
	}
708
709
	/**
710
	 * Returns the requested Jetpack API URL.
711
	 *
712
	 * @param String $relative_url the relative API path.
713
	 * @return String API URL.
714
	 */
715
	public function api_url( $relative_url ) {
716
		$api_base = Constants::get_constant( 'JETPACK__API_BASE' );
717
		$version  = Constants::get_constant( 'JETPACK__API_VERSION' );
718
719
		$api_base = $api_base ? $api_base : 'https://jetpack.wordpress.com/jetpack.';
720
		$version  = $version ? '/' . $version . '/' : '/1/';
721
722
		/**
723
		 * Filters the API URL that Jetpack uses for server communication.
724
		 *
725
		 * @since 8.0.0
726
		 *
727
		 * @param String $url the generated URL.
728
		 * @param String $relative_url the relative URL that was passed as an argument.
729
		 * @param String $api_base the API base string that is being used.
730
		 * @param String $version the version string that is being used.
731
		 */
732
		return apply_filters(
733
			'jetpack_api_url',
734
			rtrim( $api_base . $relative_url, '/\\' ) . $version,
735
			$relative_url,
736
			$api_base,
737
			$version
738
		);
739
	}
740
741
	/**
742
	 * Attempts Jetpack registration which sets up the site for connection. Should
743
	 * remain public because the call to action comes from the current site, not from
744
	 * WordPress.com.
745
	 *
746
	 * @param String $api_endpoint (optional) an API endpoint to use, defaults to 'register'.
747
	 * @return Integer zero on success, or a bitmask on failure.
748
	 */
749
	public function register( $api_endpoint = 'register' ) {
750
		add_action( 'pre_update_jetpack_option_register', array( '\\Jetpack_Options', 'delete_option' ) );
751
		$secrets = $this->generate_secrets( 'register', get_current_user_id(), 600 );
752
753
		if (
754
			empty( $secrets['secret_1'] ) ||
755
			empty( $secrets['secret_2'] ) ||
756
			empty( $secrets['exp'] )
757
		) {
758
			return new \WP_Error( 'missing_secrets' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'missing_secrets'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
759
		}
760
761
		// Better to try (and fail) to set a higher timeout than this system
762
		// supports than to have register fail for more users than it should.
763
		$timeout = $this->set_min_time_limit( 60 ) / 2;
764
765
		$gmt_offset = get_option( 'gmt_offset' );
766
		if ( ! $gmt_offset ) {
767
			$gmt_offset = 0;
768
		}
769
770
		$stats_options = get_option( 'stats_options' );
771
		$stats_id      = isset( $stats_options['blog_id'] )
772
			? $stats_options['blog_id']
773
			: null;
774
775
		/**
776
		 * Filters the request body for additional property addition.
777
		 *
778
		 * @since 7.7.0
779
		 *
780
		 * @param Array $post_data request data.
781
		 * @param Array $token_data token data.
782
		 */
783
		$body = apply_filters(
784
			'jetpack_register_request_body',
785
			array(
786
				'siteurl'         => site_url(),
787
				'home'            => home_url(),
788
				'gmt_offset'      => $gmt_offset,
789
				'timezone_string' => (string) get_option( 'timezone_string' ),
790
				'site_name'       => (string) get_option( 'blogname' ),
791
				'secret_1'        => $secrets['secret_1'],
792
				'secret_2'        => $secrets['secret_2'],
793
				'site_lang'       => get_locale(),
794
				'timeout'         => $timeout,
795
				'stats_id'        => $stats_id,
796
				'state'           => get_current_user_id(),
797
				'site_created'    => $this->get_assumed_site_creation_date(),
798
				'jetpack_version' => Constants::get_constant( 'JETPACK__VERSION' ),
799
			)
800
		);
801
802
		$args = array(
803
			'method'  => 'POST',
804
			'body'    => $body,
805
			'headers' => array(
806
				'Accept' => 'application/json',
807
			),
808
			'timeout' => $timeout,
809
		);
810
811
		$args['body'] = $this->apply_activation_source_to_args( $args['body'] );
812
813
		// TODO: fix URLs for bad hosts.
814
		$response = Client::_wp_remote_request(
815
			$this->api_url( $api_endpoint ),
816
			$args,
817
			true
818
		);
819
820
		// Make sure the response is valid and does not contain any Jetpack errors.
821
		$registration_details = $this->validate_remote_register_response( $response );
822
823
		if ( is_wp_error( $registration_details ) ) {
824
			return $registration_details;
825
		} elseif ( ! $registration_details ) {
826
			return new \WP_Error(
827
				'unknown_error',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'unknown_error'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
828
				'Unknown error registering your Jetpack site.',
829
				wp_remote_retrieve_response_code( $response )
830
			);
831
		}
832
833
		if ( empty( $registration_details->jetpack_secret ) || ! is_string( $registration_details->jetpack_secret ) ) {
834
			return new \WP_Error(
835
				'jetpack_secret',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'jetpack_secret'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
836
				'Unable to validate registration of your Jetpack site.',
837
				wp_remote_retrieve_response_code( $response )
838
			);
839
		}
840
841
		if ( isset( $registration_details->jetpack_public ) ) {
842
			$jetpack_public = (int) $registration_details->jetpack_public;
843
		} else {
844
			$jetpack_public = false;
845
		}
846
847
		\Jetpack_Options::update_options(
848
			array(
849
				'id'         => (int) $registration_details->jetpack_id,
850
				'blog_token' => (string) $registration_details->jetpack_secret,
851
				'public'     => $jetpack_public,
852
			)
853
		);
854
855
		/**
856
		 * Fires when a site is registered on WordPress.com.
857
		 *
858
		 * @since 3.7.0
859
		 *
860
		 * @param int $json->jetpack_id Jetpack Blog ID.
861
		 * @param string $json->jetpack_secret Jetpack Blog Token.
862
		 * @param int|bool $jetpack_public Is the site public.
863
		 */
864
		do_action(
865
			'jetpack_site_registered',
866
			$registration_details->jetpack_id,
867
			$registration_details->jetpack_secret,
868
			$jetpack_public
869
		);
870
871
		if ( isset( $registration_details->token ) ) {
872
			/**
873
			 * Fires when a user token is sent along with the registration data.
874
			 *
875
			 * @since 7.6.0
876
			 *
877
			 * @param object $token the administrator token for the newly registered site.
878
			 */
879
			do_action( 'jetpack_site_registered_user_token', $registration_details->token );
880
		}
881
882
		return true;
883
	}
884
885
	/**
886
	 * Takes the response from the Jetpack register new site endpoint and
887
	 * verifies it worked properly.
888
	 *
889
	 * @since 2.6
890
	 *
891
	 * @param Mixed $response the response object, or the error object.
892
	 * @return string|WP_Error A JSON object on success or Jetpack_Error on failures
893
	 **/
894
	protected function validate_remote_register_response( $response ) {
895
		if ( is_wp_error( $response ) ) {
896
			return new \WP_Error(
897
				'register_http_request_failed',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'register_http_request_failed'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
898
				$response->get_error_message()
899
			);
900
		}
901
902
		$code   = wp_remote_retrieve_response_code( $response );
903
		$entity = wp_remote_retrieve_body( $response );
904
905
		if ( $entity ) {
906
			$registration_response = json_decode( $entity );
907
		} else {
908
			$registration_response = false;
909
		}
910
911
		$code_type = intval( $code / 100 );
912
		if ( 5 === $code_type ) {
913
			return new \WP_Error( 'wpcom_5??', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'wpcom_5??'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
914
		} elseif ( 408 === $code ) {
915
			return new \WP_Error( 'wpcom_408', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'wpcom_408'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
916
		} elseif ( ! empty( $registration_response->error ) ) {
917
			if (
918
				'xml_rpc-32700' === $registration_response->error
919
				&& ! function_exists( 'xml_parser_create' )
920
			) {
921
				$error_description = __( "PHP's XML extension is not available. Jetpack requires the XML extension to communicate with WordPress.com. Please contact your hosting provider to enable PHP's XML extension.", 'jetpack' );
922
			} else {
923
				$error_description = isset( $registration_response->error_description )
924
					? (string) $registration_response->error_description
925
					: '';
926
			}
927
928
			return new \WP_Error(
929
				(string) $registration_response->error,
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with (string) $registration_response->error.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
930
				$error_description,
931
				$code
932
			);
933
		} elseif ( 200 !== $code ) {
934
			return new \WP_Error( 'wpcom_bad_response', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'wpcom_bad_response'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
935
		}
936
937
		// Jetpack ID error block.
938
		if ( empty( $registration_response->jetpack_id ) ) {
939
			return new \WP_Error(
940
				'jetpack_id',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'jetpack_id'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
941
				/* translators: %s is an error message string */
942
				sprintf( __( 'Error Details: Jetpack ID is empty. Do not publicly post this error message! %s', 'jetpack' ), $entity ),
943
				$entity
944
			);
945
		} elseif ( ! is_scalar( $registration_response->jetpack_id ) ) {
946
			return new \WP_Error(
947
				'jetpack_id',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'jetpack_id'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
948
				/* translators: %s is an error message string */
949
				sprintf( __( 'Error Details: Jetpack ID is not a scalar. Do not publicly post this error message! %s', 'jetpack' ), $entity ),
950
				$entity
951
			);
952 View Code Duplication
		} elseif ( preg_match( '/[^0-9]/', $registration_response->jetpack_id ) ) {
953
			return new \WP_Error(
954
				'jetpack_id',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'jetpack_id'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
955
				/* translators: %s is an error message string */
956
				sprintf( __( 'Error Details: Jetpack ID begins with a numeral. Do not publicly post this error message! %s', 'jetpack' ), $entity ),
957
				$entity
958
			);
959
		}
960
961
		return $registration_response;
962
	}
963
964
	/**
965
	 * Adds a used nonce to a list of known nonces.
966
	 *
967
	 * @param int    $timestamp the current request timestamp.
968
	 * @param string $nonce the nonce value.
969
	 * @return bool whether the nonce is unique or not.
970
	 */
971
	public function add_nonce( $timestamp, $nonce ) {
972
		global $wpdb;
973
		static $nonces_used_this_request = array();
974
975
		if ( isset( $nonces_used_this_request[ "$timestamp:$nonce" ] ) ) {
976
			return $nonces_used_this_request[ "$timestamp:$nonce" ];
977
		}
978
979
		// This should always have gone through Jetpack_Signature::sign_request() first to check $timestamp an $nonce.
980
		$timestamp = (int) $timestamp;
981
		$nonce     = esc_sql( $nonce );
982
983
		// Raw query so we can avoid races: add_option will also update.
984
		$show_errors = $wpdb->show_errors( false );
985
986
		$old_nonce = $wpdb->get_row(
987
			$wpdb->prepare( "SELECT * FROM `$wpdb->options` WHERE option_name = %s", "jetpack_nonce_{$timestamp}_{$nonce}" )
988
		);
989
990
		if ( is_null( $old_nonce ) ) {
991
			$return = $wpdb->query(
992
				$wpdb->prepare(
993
					"INSERT INTO `$wpdb->options` (`option_name`, `option_value`, `autoload`) VALUES (%s, %s, %s)",
994
					"jetpack_nonce_{$timestamp}_{$nonce}",
995
					time(),
996
					'no'
997
				)
998
			);
999
		} else {
1000
			$return = false;
1001
		}
1002
1003
		$wpdb->show_errors( $show_errors );
1004
1005
		$nonces_used_this_request[ "$timestamp:$nonce" ] = $return;
1006
1007
		return $return;
1008
	}
1009
1010
	/**
1011
	 * Cleans nonces that were saved when calling ::add_nonce.
1012
	 *
1013
	 * @todo Properly prepare the query before executing it.
1014
	 *
1015
	 * @param bool $all whether to clean even non-expired nonces.
1016
	 */
1017
	public function clean_nonces( $all = false ) {
1018
		global $wpdb;
1019
1020
		$sql      = "DELETE FROM `$wpdb->options` WHERE `option_name` LIKE %s";
1021
		$sql_args = array( $wpdb->esc_like( 'jetpack_nonce_' ) . '%' );
1022
1023
		if ( true !== $all ) {
1024
			$sql       .= ' AND CAST( `option_value` AS UNSIGNED ) < %d';
1025
			$sql_args[] = time() - 3600;
1026
		}
1027
1028
		$sql .= ' ORDER BY `option_id` LIMIT 100';
1029
1030
		$sql = $wpdb->prepare( $sql, $sql_args ); // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
1031
1032
		for ( $i = 0; $i < 1000; $i++ ) {
1033
			if ( ! $wpdb->query( $sql ) ) { // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
1034
				break;
1035
			}
1036
		}
1037
	}
1038
1039
	/**
1040
	 * Builds the timeout limit for queries talking with the wpcom servers.
1041
	 *
1042
	 * Based on local php max_execution_time in php.ini
1043
	 *
1044
	 * @since 5.4
1045
	 * @return int
1046
	 **/
1047
	public function get_max_execution_time() {
1048
		$timeout = (int) ini_get( 'max_execution_time' );
1049
1050
		// Ensure exec time set in php.ini.
1051
		if ( ! $timeout ) {
1052
			$timeout = 30;
1053
		}
1054
		return $timeout;
1055
	}
1056
1057
	/**
1058
	 * Sets a minimum request timeout, and returns the current timeout
1059
	 *
1060
	 * @since 5.4
1061
	 * @param Integer $min_timeout the minimum timeout value.
1062
	 **/
1063 View Code Duplication
	public function set_min_time_limit( $min_timeout ) {
1064
		$timeout = $this->get_max_execution_time();
1065
		if ( $timeout < $min_timeout ) {
1066
			$timeout = $min_timeout;
1067
			set_time_limit( $timeout );
1068
		}
1069
		return $timeout;
1070
	}
1071
1072
	/**
1073
	 * Get our assumed site creation date.
1074
	 * Calculated based on the earlier date of either:
1075
	 * - Earliest admin user registration date.
1076
	 * - Earliest date of post of any post type.
1077
	 *
1078
	 * @since 7.2.0
1079
	 *
1080
	 * @return string Assumed site creation date and time.
1081
	 */
1082
	public function get_assumed_site_creation_date() {
1083
		$cached_date = get_transient( 'jetpack_assumed_site_creation_date' );
1084
		if ( ! empty( $cached_date ) ) {
1085
			return $cached_date;
1086
		}
1087
1088
		$earliest_registered_users  = get_users(
1089
			array(
1090
				'role'    => 'administrator',
1091
				'orderby' => 'user_registered',
1092
				'order'   => 'ASC',
1093
				'fields'  => array( 'user_registered' ),
1094
				'number'  => 1,
1095
			)
1096
		);
1097
		$earliest_registration_date = $earliest_registered_users[0]->user_registered;
1098
1099
		$earliest_posts = get_posts(
1100
			array(
1101
				'posts_per_page' => 1,
1102
				'post_type'      => 'any',
1103
				'post_status'    => 'any',
1104
				'orderby'        => 'date',
1105
				'order'          => 'ASC',
1106
			)
1107
		);
1108
1109
		// If there are no posts at all, we'll count only on user registration date.
1110
		if ( $earliest_posts ) {
1111
			$earliest_post_date = $earliest_posts[0]->post_date;
1112
		} else {
1113
			$earliest_post_date = PHP_INT_MAX;
1114
		}
1115
1116
		$assumed_date = min( $earliest_registration_date, $earliest_post_date );
1117
		set_transient( 'jetpack_assumed_site_creation_date', $assumed_date );
1118
1119
		return $assumed_date;
1120
	}
1121
1122
	/**
1123
	 * Adds the activation source string as a parameter to passed arguments.
1124
	 *
1125
	 * @todo Refactor to use rawurlencode() instead of urlencode().
1126
	 *
1127
	 * @param Array $args arguments that need to have the source added.
1128
	 * @return Array $amended arguments.
1129
	 */
1130 View Code Duplication
	public static function apply_activation_source_to_args( $args ) {
1131
		list( $activation_source_name, $activation_source_keyword ) = get_option( 'jetpack_activation_source' );
1132
1133
		if ( $activation_source_name ) {
1134
			// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.urlencode_urlencode
1135
			$args['_as'] = urlencode( $activation_source_name );
1136
		}
1137
1138
		if ( $activation_source_keyword ) {
1139
			// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.urlencode_urlencode
1140
			$args['_ak'] = urlencode( $activation_source_keyword );
1141
		}
1142
1143
		return $args;
1144
	}
1145
1146
	/**
1147
	 * Returns the callable that would be used to generate secrets.
1148
	 *
1149
	 * @return Callable a function that returns a secure string to be used as a secret.
1150
	 */
1151
	protected function get_secret_callable() {
1152
		if ( ! isset( $this->secret_callable ) ) {
1153
			/**
1154
			 * Allows modification of the callable that is used to generate connection secrets.
1155
			 *
1156
			 * @param Callable a function or method that returns a secret string.
1157
			 */
1158
			$this->secret_callable = apply_filters( 'jetpack_connection_secret_generator', 'wp_generate_password' );
1159
		}
1160
1161
		return $this->secret_callable;
1162
	}
1163
1164
	/**
1165
	 * Generates two secret tokens and the end of life timestamp for them.
1166
	 *
1167
	 * @param String  $action  The action name.
1168
	 * @param Integer $user_id The user identifier.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user_id not be false|integer?

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...
1169
	 * @param Integer $exp     Expiration time in seconds.
1170
	 */
1171
	public function generate_secrets( $action, $user_id = false, $exp = 600 ) {
1172
		if ( false === $user_id ) {
1173
			$user_id = get_current_user_id();
1174
		}
1175
1176
		$callable = $this->get_secret_callable();
1177
1178
		$secrets = \Jetpack_Options::get_raw_option(
1179
			self::SECRETS_OPTION_NAME,
1180
			array()
1181
		);
1182
1183
		$secret_name = 'jetpack_' . $action . '_' . $user_id;
1184
1185
		if (
1186
			isset( $secrets[ $secret_name ] ) &&
1187
			$secrets[ $secret_name ]['exp'] > time()
1188
		) {
1189
			return $secrets[ $secret_name ];
1190
		}
1191
1192
		$secret_value = array(
1193
			'secret_1' => call_user_func( $callable ),
1194
			'secret_2' => call_user_func( $callable ),
1195
			'exp'      => time() + $exp,
1196
		);
1197
1198
		$secrets[ $secret_name ] = $secret_value;
1199
1200
		\Jetpack_Options::update_raw_option( self::SECRETS_OPTION_NAME, $secrets );
1201
		return $secrets[ $secret_name ];
1202
	}
1203
1204
	/**
1205
	 * Returns two secret tokens and the end of life timestamp for them.
1206
	 *
1207
	 * @param String  $action  The action name.
1208
	 * @param Integer $user_id The user identifier.
1209
	 * @return string|array an array of secrets or an error string.
1210
	 */
1211
	public function get_secrets( $action, $user_id ) {
1212
		$secret_name = 'jetpack_' . $action . '_' . $user_id;
1213
		$secrets     = \Jetpack_Options::get_raw_option(
1214
			self::SECRETS_OPTION_NAME,
1215
			array()
1216
		);
1217
1218
		if ( ! isset( $secrets[ $secret_name ] ) ) {
1219
			return self::SECRETS_MISSING;
1220
		}
1221
1222
		if ( $secrets[ $secret_name ]['exp'] < time() ) {
1223
			$this->delete_secrets( $action, $user_id );
1224
			return self::SECRETS_EXPIRED;
1225
		}
1226
1227
		return $secrets[ $secret_name ];
1228
	}
1229
1230
	/**
1231
	 * Deletes secret tokens in case they, for example, have expired.
1232
	 *
1233
	 * @param String  $action  The action name.
1234
	 * @param Integer $user_id The user identifier.
1235
	 */
1236
	public function delete_secrets( $action, $user_id ) {
1237
		$secret_name = 'jetpack_' . $action . '_' . $user_id;
1238
		$secrets     = \Jetpack_Options::get_raw_option(
1239
			self::SECRETS_OPTION_NAME,
1240
			array()
1241
		);
1242
		if ( isset( $secrets[ $secret_name ] ) ) {
1243
			unset( $secrets[ $secret_name ] );
1244
			\Jetpack_Options::update_raw_option( self::SECRETS_OPTION_NAME, $secrets );
1245
		}
1246
	}
1247
1248
	/**
1249
	 * Responds to a WordPress.com call to register the current site.
1250
	 * Should be changed to protected.
1251
	 *
1252
	 * @param array $registration_data Array of [ secret_1, user_id ].
1253
	 */
1254
	public function handle_registration( array $registration_data ) {
1255
		list( $registration_secret_1, $registration_user_id ) = $registration_data;
1256
		if ( empty( $registration_user_id ) ) {
1257
			return new \WP_Error( 'registration_state_invalid', __( 'Invalid Registration State', 'jetpack' ), 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'registration_state_invalid'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1258
		}
1259
1260
		return $this->verify_secrets( 'register', $registration_secret_1, (int) $registration_user_id );
1261
	}
1262
1263
	/**
1264
	 * Verify a Previously Generated Secret.
1265
	 *
1266
	 * @param string $action   The type of secret to verify.
1267
	 * @param string $secret_1 The secret string to compare to what is stored.
1268
	 * @param int    $user_id  The user ID of the owner of the secret.
1269
	 * @return \WP_Error|string WP_Error on failure, secret_2 on success.
1270
	 */
1271
	public function verify_secrets( $action, $secret_1, $user_id ) {
1272
		$allowed_actions = array( 'register', 'authorize', 'publicize' );
1273
		if ( ! in_array( $action, $allowed_actions, true ) ) {
1274
			return new \WP_Error( 'unknown_verification_action', 'Unknown Verification Action', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'unknown_verification_action'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1275
		}
1276
1277
		$user = get_user_by( 'id', $user_id );
1278
1279
		/**
1280
		 * We've begun verifying the previously generated secret.
1281
		 *
1282
		 * @since 7.5.0
1283
		 *
1284
		 * @param string   $action The type of secret to verify.
1285
		 * @param \WP_User $user The user object.
1286
		 */
1287
		do_action( 'jetpack_verify_secrets_begin', $action, $user );
1288
1289
		$return_error = function( \WP_Error $error ) use ( $action, $user ) {
1290
			/**
1291
			 * Verifying of the previously generated secret has failed.
1292
			 *
1293
			 * @since 7.5.0
1294
			 *
1295
			 * @param string    $action  The type of secret to verify.
1296
			 * @param \WP_User  $user The user object.
1297
			 * @param \WP_Error $error The error object.
1298
			 */
1299
			do_action( 'jetpack_verify_secrets_fail', $action, $user, $error );
1300
1301
			return $error;
1302
		};
1303
1304
		$stored_secrets = $this->get_secrets( $action, $user_id );
1305
		$this->delete_secrets( $action, $user_id );
1306
1307
		$error = null;
1308
		if ( empty( $secret_1 ) ) {
1309
			$error = $return_error(
1310
				new \WP_Error(
1311
					'verify_secret_1_missing',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secret_1_missing'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1312
					/* translators: "%s" is the name of a paramter. It can be either "secret_1" or "state". */
1313
					sprintf( __( 'The required "%s" parameter is missing.', 'jetpack' ), 'secret_1' ),
1314
					400
1315
				)
1316
			);
1317
		} elseif ( ! is_string( $secret_1 ) ) {
1318
			$error = $return_error(
1319
				new \WP_Error(
1320
					'verify_secret_1_malformed',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secret_1_malformed'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1321
					/* translators: "%s" is the name of a paramter. It can be either "secret_1" or "state". */
1322
					sprintf( __( 'The required "%s" parameter is malformed.', 'jetpack' ), 'secret_1' ),
1323
					400
1324
				)
1325
			);
1326
		} elseif ( empty( $user_id ) ) {
1327
			// $user_id is passed around during registration as "state".
1328
			$error = $return_error(
1329
				new \WP_Error(
1330
					'state_missing',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'state_missing'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1331
					/* translators: "%s" is the name of a paramter. It can be either "secret_1" or "state". */
1332
					sprintf( __( 'The required "%s" parameter is missing.', 'jetpack' ), 'state' ),
1333
					400
1334
				)
1335
			);
1336
		} elseif ( ! ctype_digit( (string) $user_id ) ) {
1337
			$error = $return_error(
1338
				new \WP_Error(
1339
					'state_malformed',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'state_malformed'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1340
					/* translators: "%s" is the name of a paramter. It can be either "secret_1" or "state". */
1341
					sprintf( __( 'The required "%s" parameter is malformed.', 'jetpack' ), 'state' ),
1342
					400
1343
				)
1344
			);
1345
		} elseif ( self::SECRETS_MISSING === $stored_secrets ) {
1346
			$error = $return_error(
1347
				new \WP_Error(
1348
					'verify_secrets_missing',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secrets_missing'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1349
					__( 'Verification secrets not found', 'jetpack' ),
1350
					400
1351
				)
1352
			);
1353
		} elseif ( self::SECRETS_EXPIRED === $stored_secrets ) {
1354
			$error = $return_error(
1355
				new \WP_Error(
1356
					'verify_secrets_expired',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secrets_expired'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1357
					__( 'Verification took too long', 'jetpack' ),
1358
					400
1359
				)
1360
			);
1361
		} elseif ( ! $stored_secrets ) {
1362
			$error = $return_error(
1363
				new \WP_Error(
1364
					'verify_secrets_empty',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secrets_empty'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1365
					__( 'Verification secrets are empty', 'jetpack' ),
1366
					400
1367
				)
1368
			);
1369
		} elseif ( is_wp_error( $stored_secrets ) ) {
1370
			$stored_secrets->add_data( 400 );
0 ignored issues
show
Bug introduced by
The method add_data cannot be called on $stored_secrets (of type string|array).

Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects.

Loading history...
1371
			$error = $return_error( $stored_secrets );
1372
		} elseif ( empty( $stored_secrets['secret_1'] ) || empty( $stored_secrets['secret_2'] ) || empty( $stored_secrets['exp'] ) ) {
1373
			$error = $return_error(
1374
				new \WP_Error(
1375
					'verify_secrets_incomplete',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secrets_incomplete'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1376
					__( 'Verification secrets are incomplete', 'jetpack' ),
1377
					400
1378
				)
1379
			);
1380
		} elseif ( ! hash_equals( $secret_1, $stored_secrets['secret_1'] ) ) {
1381
			$error = $return_error(
1382
				new \WP_Error(
1383
					'verify_secrets_mismatch',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'verify_secrets_mismatch'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1384
					__( 'Secret mismatch', 'jetpack' ),
1385
					400
1386
				)
1387
			);
1388
		}
1389
1390
		// Something went wrong during the checks, returning the error.
1391
		if ( ! empty( $error ) ) {
1392
			return $error;
1393
		}
1394
1395
		/**
1396
		 * We've succeeded at verifying the previously generated secret.
1397
		 *
1398
		 * @since 7.5.0
1399
		 *
1400
		 * @param string   $action The type of secret to verify.
1401
		 * @param \WP_User $user The user object.
1402
		 */
1403
		do_action( 'jetpack_verify_secrets_success', $action, $user );
1404
1405
		return $stored_secrets['secret_2'];
1406
	}
1407
1408
	/**
1409
	 * Responds to a WordPress.com call to authorize the current user.
1410
	 * Should be changed to protected.
1411
	 */
1412
	public function handle_authorization() {
1413
1414
	}
1415
1416
	/**
1417
	 * Obtains the auth token.
1418
	 *
1419
	 * @param array $data The request data.
1420
	 * @return object|\WP_Error Returns the auth token on success.
1421
	 *                          Returns a \WP_Error on failure.
1422
	 */
1423
	public function get_token( $data ) {
1424
		$roles = new Roles();
1425
		$role  = $roles->translate_current_user_to_role();
1426
1427
		if ( ! $role ) {
1428
			return new \WP_Error( 'role', __( 'An administrator for this blog must set up the Jetpack connection.', 'jetpack' ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'role'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1429
		}
1430
1431
		$client_secret = $this->get_access_token();
1432
		if ( ! $client_secret ) {
1433
			return new \WP_Error( 'client_secret', __( 'You need to register your Jetpack before connecting it.', 'jetpack' ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'client_secret'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1434
		}
1435
1436
		/**
1437
		 * Filter the URL of the first time the user gets redirected back to your site for connection
1438
		 * data processing.
1439
		 *
1440
		 * @since 8.0.0
1441
		 *
1442
		 * @param string $redirect_url Defaults to the site admin URL.
1443
		 */
1444
		$processing_url = apply_filters( 'jetpack_token_processing_url', admin_url( 'admin.php' ) );
1445
1446
		$redirect = isset( $data['redirect'] ) ? esc_url_raw( (string) $data['redirect'] ) : '';
1447
1448
		/**
1449
		* Filter the URL to redirect the user back to when the authentication process
1450
		* is complete.
1451
		*
1452
		* @since 8.0.0
1453
		*
1454
		* @param string $redirect_url Defaults to the site URL.
1455
		*/
1456
		$redirect = apply_filters( 'jetpack_token_redirect_url', $redirect );
1457
1458
		$redirect_uri = ( 'calypso' === $data['auth_type'] )
1459
			? $data['redirect_uri']
1460
			: add_query_arg(
1461
				array(
1462
					'action'   => 'authorize',
1463
					'_wpnonce' => wp_create_nonce( "jetpack-authorize_{$role}_{$redirect}" ),
1464
					'redirect' => $redirect ? rawurlencode( $redirect ) : false,
1465
				),
1466
				esc_url( $processing_url )
1467
			);
1468
1469
		/**
1470
		 * Filters the token request data.
1471
		 *
1472
		 * @since 8.0.0
1473
		 *
1474
		 * @param Array $request_data request data.
1475
		 */
1476
		$body = apply_filters(
1477
			'jetpack_token_request_body',
1478
			array(
1479
				'client_id'     => \Jetpack_Options::get_option( 'id' ),
1480
				'client_secret' => $client_secret->secret,
1481
				'grant_type'    => 'authorization_code',
1482
				'code'          => $data['code'],
1483
				'redirect_uri'  => $redirect_uri,
1484
			)
1485
		);
1486
1487
		$args = array(
1488
			'method'  => 'POST',
1489
			'body'    => $body,
1490
			'headers' => array(
1491
				'Accept' => 'application/json',
1492
			),
1493
		);
1494
1495
		$response = Client::_wp_remote_request( Utils::fix_url_for_bad_hosts( $this->api_url( 'token' ) ), $args );
1496
1497
		if ( is_wp_error( $response ) ) {
1498
			return new \WP_Error( 'token_http_request_failed', $response->get_error_message() );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'token_http_request_failed'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1499
		}
1500
1501
		$code   = wp_remote_retrieve_response_code( $response );
1502
		$entity = wp_remote_retrieve_body( $response );
1503
1504
		if ( $entity ) {
1505
			$json = json_decode( $entity );
1506
		} else {
1507
			$json = false;
1508
		}
1509
1510
		if ( 200 !== $code || ! empty( $json->error ) ) {
1511
			if ( empty( $json->error ) ) {
1512
				return new \WP_Error( 'unknown', '', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'unknown'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1513
			}
1514
1515
			$error_description = isset( $json->error_description ) ? sprintf( __( 'Error Details: %s', 'jetpack' ), (string) $json->error_description ) : '';
1516
1517
			return new \WP_Error( (string) $json->error, $error_description, $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with (string) $json->error.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1518
		}
1519
1520
		if ( empty( $json->access_token ) || ! is_scalar( $json->access_token ) ) {
1521
			return new \WP_Error( 'access_token', '', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'access_token'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1522
		}
1523
1524
		if ( empty( $json->token_type ) || 'X_JETPACK' !== strtoupper( $json->token_type ) ) {
1525
			return new \WP_Error( 'token_type', '', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'token_type'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1526
		}
1527
1528
		if ( empty( $json->scope ) ) {
1529
			return new \WP_Error( 'scope', 'No Scope', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'scope'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1530
		}
1531
1532
		@list( $role, $hmac ) = explode( ':', $json->scope );
0 ignored issues
show
Security Best Practice introduced by
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...
1533
		if ( empty( $role ) || empty( $hmac ) ) {
1534
			return new \WP_Error( 'scope', 'Malformed Scope', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'scope'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1535
		}
1536
1537
		if ( $this->sign_role( $role ) !== $json->scope ) {
1538
			return new \WP_Error( 'scope', 'Invalid Scope', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'scope'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1539
		}
1540
1541
		$cap = $roles->translate_role_to_cap( $role );
1542
		if ( ! $cap ) {
1543
			return new \WP_Error( 'scope', 'No Cap', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'scope'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1544
		}
1545
1546
		if ( ! current_user_can( $cap ) ) {
1547
			return new \WP_Error( 'scope', 'current_user_cannot', $code );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'scope'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1548
		}
1549
1550
		/**
1551
		 * Fires after user has successfully received an auth token.
1552
		 *
1553
		 * @since 3.9.0
1554
		 */
1555
		do_action( 'jetpack_user_authorized' );
1556
1557
		return (string) $json->access_token;
1558
	}
1559
1560
	/**
1561
	 * Builds a URL to the Jetpack connection auth page.
1562
	 *
1563
	 * @param WP_User $user (optional) defaults to the current logged in user.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user not be WP_User|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...
1564
	 * @param String  $redirect (optional) a redirect URL to use instead of the default.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $redirect 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...
1565
	 * @return string Connect URL.
1566
	 */
1567
	public function get_authorization_url( $user = null, $redirect = null ) {
1568
1569
		if ( empty( $user ) ) {
1570
			$user = wp_get_current_user();
1571
		}
1572
1573
		$roles       = new Roles();
1574
		$role        = $roles->translate_user_to_role( $user );
1575
		$signed_role = $this->sign_role( $role );
1576
1577
		/**
1578
		 * Filter the URL of the first time the user gets redirected back to your site for connection
1579
		 * data processing.
1580
		 *
1581
		 * @since 8.0.0
1582
		 *
1583
		 * @param string $redirect_url Defaults to the site admin URL.
1584
		 */
1585
		$processing_url = apply_filters( 'jetpack_connect_processing_url', admin_url( 'admin.php' ) );
1586
1587
		/**
1588
		 * Filter the URL to redirect the user back to when the authorization process
1589
		 * is complete.
1590
		 *
1591
		 * @since 8.0.0
1592
		 *
1593
		 * @param string $redirect_url Defaults to the site URL.
1594
		 */
1595
		$redirect = apply_filters( 'jetpack_connect_redirect_url', $redirect );
1596
1597
		$secrets = $this->generate_secrets( 'authorize', $user->ID, 2 * HOUR_IN_SECONDS );
1598
1599
		/**
1600
		 * Filter the type of authorization.
1601
		 * 'calypso' completes authorization on wordpress.com/jetpack/connect
1602
		 * while 'jetpack' ( or any other value ) completes the authorization at jetpack.wordpress.com.
1603
		 *
1604
		 * @since 4.3.3
1605
		 *
1606
		 * @param string $auth_type Defaults to 'calypso', can also be 'jetpack'.
1607
		 */
1608
		$auth_type = apply_filters( 'jetpack_auth_type', 'calypso' );
1609
1610
		/**
1611
		 * Filters the user connection request data for additional property addition.
1612
		 *
1613
		 * @since 8.0.0
1614
		 *
1615
		 * @param Array $request_data request data.
1616
		 */
1617
		$body = apply_filters(
1618
			'jetpack_connect_request_body',
1619
			array(
1620
				'response_type' => 'code',
1621
				'client_id'     => \Jetpack_Options::get_option( 'id' ),
1622
				'redirect_uri'  => add_query_arg(
1623
					array(
1624
						'action'   => 'authorize',
1625
						'_wpnonce' => wp_create_nonce( "jetpack-authorize_{$role}_{$redirect}" ),
1626
						'redirect' => rawurlencode( $redirect ),
1627
					),
1628
					esc_url( $processing_url )
1629
				),
1630
				'state'         => $user->ID,
1631
				'scope'         => $signed_role,
1632
				'user_email'    => $user->user_email,
1633
				'user_login'    => $user->user_login,
1634
				'is_active'     => $this->is_active(),
1635
				'jp_version'    => Constants::get_constant( 'JETPACK__VERSION' ),
1636
				'auth_type'     => $auth_type,
1637
				'secret'        => $secrets['secret_1'],
1638
				'blogname'      => get_option( 'blogname' ),
1639
				'site_url'      => site_url(),
1640
				'home_url'      => home_url(),
1641
				'site_icon'     => get_site_icon_url(),
1642
				'site_lang'     => get_locale(),
1643
				'site_created'  => $this->get_assumed_site_creation_date(),
1644
			)
1645
		);
1646
1647
		$body = $this->apply_activation_source_to_args( urlencode_deep( $body ) );
1648
1649
		$api_url = $this->api_url( 'authorize' );
1650
1651
		return add_query_arg( $body, $api_url );
1652
	}
1653
1654
	/**
1655
	 * Authorizes the user by obtaining and storing the user token.
1656
	 *
1657
	 * @param array $data The request data.
1658
	 * @return string|\WP_Error Returns a string on success.
1659
	 *                          Returns a \WP_Error on failure.
1660
	 */
1661
	public function authorize( $data = array() ) {
1662
		/**
1663
		 * Action fired when user authorization starts.
1664
		 *
1665
		 * @since 8.0.0
1666
		 */
1667
		do_action( 'jetpack_authorize_starting' );
1668
1669
		$roles = new Roles();
1670
		$role  = $roles->translate_current_user_to_role();
1671
1672
		if ( ! $role ) {
1673
			return new \WP_Error( 'no_role', 'Invalid request.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_role'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1674
		}
1675
1676
		$cap = $roles->translate_role_to_cap( $role );
1677
		if ( ! $cap ) {
1678
			return new \WP_Error( 'no_cap', 'Invalid request.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_cap'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1679
		}
1680
1681
		if ( ! empty( $data['error'] ) ) {
1682
			return new \WP_Error( $data['error'], 'Error included in the request.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with $data['error'].

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1683
		}
1684
1685
		if ( ! isset( $data['state'] ) ) {
1686
			return new \WP_Error( 'no_state', 'Request must include state.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_state'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1687
		}
1688
1689
		if ( ! ctype_digit( $data['state'] ) ) {
1690
			return new \WP_Error( $data['error'], 'State must be an integer.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with $data['error'].

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1691
		}
1692
1693
		$current_user_id = get_current_user_id();
1694
		if ( $current_user_id !== (int) $data['state'] ) {
1695
			return new \WP_Error( 'wrong_state', 'State does not match current user.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'wrong_state'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1696
		}
1697
1698
		if ( empty( $data['code'] ) ) {
1699
			return new \WP_Error( 'no_code', 'Request must include an authorization code.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_code'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1700
		}
1701
1702
		$token = $this->get_token( $data );
1703
1704 View Code Duplication
		if ( is_wp_error( $token ) ) {
1705
			$code = $token->get_error_code();
0 ignored issues
show
Bug introduced by
The method get_error_code() does not seem to exist on object<WP_Error>.

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
1706
			if ( empty( $code ) ) {
1707
				$code = 'invalid_token';
1708
			}
1709
			return new \WP_Error( $code, $token->get_error_message(), 400 );
0 ignored issues
show
Bug introduced by
The method get_error_message() does not seem to exist on object<WP_Error>.

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with $code.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1710
		}
1711
1712
		if ( ! $token ) {
1713
			return new \WP_Error( 'no_token', 'Error generating token.', 400 );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_token'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1714
		}
1715
1716
		$is_master_user = ! $this->is_active();
1717
1718
		Utils::update_user_token( $current_user_id, sprintf( '%s.%d', $token, $current_user_id ), $is_master_user );
1719
1720
		if ( ! $is_master_user ) {
1721
			/**
1722
			 * Action fired when a secondary user has been authorized.
1723
			 *
1724
			 * @since 8.0.0
1725
			 */
1726
			do_action( 'jetpack_authorize_ending_linked' );
1727
			return 'linked';
1728
		}
1729
1730
		/**
1731
		 * Action fired when the master user has been authorized.
1732
		 *
1733
		 * @since 8.0.0
1734
		 *
1735
		 * @param array $data The request data.
1736
		 */
1737
		do_action( 'jetpack_authorize_ending_authorized', $data );
1738
1739
		return 'authorized';
1740
	}
1741
1742
	/**
1743
	 * Disconnects from the Jetpack servers.
1744
	 * Forgets all connection details and tells the Jetpack servers to do the same.
1745
	 */
1746
	public function disconnect_site() {
1747
1748
	}
1749
1750
	/**
1751
	 * The Base64 Encoding of the SHA1 Hash of the Input.
1752
	 *
1753
	 * @param string $text The string to hash.
1754
	 * @return string
1755
	 */
1756
	public function sha1_base64( $text ) {
1757
		return base64_encode( sha1( $text, true ) ); // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.obfuscation_base64_encode
1758
	}
1759
1760
	/**
1761
	 * This function mirrors Jetpack_Data::is_usable_domain() in the WPCOM codebase.
1762
	 *
1763
	 * @param string $domain The domain to check.
1764
	 *
1765
	 * @return bool|WP_Error
1766
	 */
1767
	public function is_usable_domain( $domain ) {
1768
1769
		// If it's empty, just fail out.
1770
		if ( ! $domain ) {
1771
			return new \WP_Error(
1772
				'fail_domain_empty',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'fail_domain_empty'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1773
				/* translators: %1$s is a domain name. */
1774
				sprintf( __( 'Domain `%1$s` just failed is_usable_domain check as it is empty.', 'jetpack' ), $domain )
1775
			);
1776
		}
1777
1778
		/**
1779
		 * Skips the usuable domain check when connecting a site.
1780
		 *
1781
		 * Allows site administrators with domains that fail gethostname-based checks to pass the request to WP.com
1782
		 *
1783
		 * @since 4.1.0
1784
		 *
1785
		 * @param bool If the check should be skipped. Default false.
1786
		 */
1787
		if ( apply_filters( 'jetpack_skip_usuable_domain_check', false ) ) {
1788
			return true;
1789
		}
1790
1791
		// None of the explicit localhosts.
1792
		$forbidden_domains = array(
1793
			'wordpress.com',
1794
			'localhost',
1795
			'localhost.localdomain',
1796
			'127.0.0.1',
1797
			'local.wordpress.test',         // VVV pattern.
1798
			'local.wordpress-trunk.test',   // VVV pattern.
1799
			'src.wordpress-develop.test',   // VVV pattern.
1800
			'build.wordpress-develop.test', // VVV pattern.
1801
		);
1802 View Code Duplication
		if ( in_array( $domain, $forbidden_domains, true ) ) {
1803
			return new \WP_Error(
1804
				'fail_domain_forbidden',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'fail_domain_forbidden'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1805
				sprintf(
1806
					/* translators: %1$s is a domain name. */
1807
					__(
1808
						'Domain `%1$s` just failed is_usable_domain check as it is in the forbidden array.',
1809
						'jetpack'
1810
					),
1811
					$domain
1812
				)
1813
			);
1814
		}
1815
1816
		// No .test or .local domains.
1817 View Code Duplication
		if ( preg_match( '#\.(test|local)$#i', $domain ) ) {
1818
			return new \WP_Error(
1819
				'fail_domain_tld',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'fail_domain_tld'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1820
				sprintf(
1821
					/* translators: %1$s is a domain name. */
1822
					__(
1823
						'Domain `%1$s` just failed is_usable_domain check as it uses an invalid top level domain.',
1824
						'jetpack'
1825
					),
1826
					$domain
1827
				)
1828
			);
1829
		}
1830
1831
		// No WPCOM subdomains.
1832 View Code Duplication
		if ( preg_match( '#\.WordPress\.com$#i', $domain ) ) {
1833
			return new \WP_Error(
1834
				'fail_subdomain_wpcom',
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'fail_subdomain_wpcom'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1835
				sprintf(
1836
					/* translators: %1$s is a domain name. */
1837
					__(
1838
						'Domain `%1$s` just failed is_usable_domain check as it is a subdomain of WordPress.com.',
1839
						'jetpack'
1840
					),
1841
					$domain
1842
				)
1843
			);
1844
		}
1845
1846
		// If PHP was compiled without support for the Filter module (very edge case).
1847
		if ( ! function_exists( 'filter_var' ) ) {
1848
			// Just pass back true for now, and let wpcom sort it out.
1849
			return true;
1850
		}
1851
1852
		return true;
1853
	}
1854
1855
	/**
1856
	 * Gets the requested token.
1857
	 *
1858
	 * Tokens are one of two types:
1859
	 * 1. Blog Tokens: These are the "main" tokens. Each site typically has one Blog Token,
1860
	 *    though some sites can have multiple "Special" Blog Tokens (see below). These tokens
1861
	 *    are not associated with a user account. They represent the site's connection with
1862
	 *    the Jetpack servers.
1863
	 * 2. User Tokens: These are "sub-"tokens. Each connected user account has one User Token.
1864
	 *
1865
	 * All tokens look like "{$token_key}.{$private}". $token_key is a public ID for the
1866
	 * token, and $private is a secret that should never be displayed anywhere or sent
1867
	 * over the network; it's used only for signing things.
1868
	 *
1869
	 * Blog Tokens can be "Normal" or "Special".
1870
	 * * Normal: The result of a normal connection flow. They look like
1871
	 *   "{$random_string_1}.{$random_string_2}"
1872
	 *   That is, $token_key and $private are both random strings.
1873
	 *   Sites only have one Normal Blog Token. Normal Tokens are found in either
1874
	 *   Jetpack_Options::get_option( 'blog_token' ) (usual) or the JETPACK_BLOG_TOKEN
1875
	 *   constant (rare).
1876
	 * * Special: A connection token for sites that have gone through an alternative
1877
	 *   connection flow. They look like:
1878
	 *   ";{$special_id}{$special_version};{$wpcom_blog_id};.{$random_string}"
1879
	 *   That is, $private is a random string and $token_key has a special structure with
1880
	 *   lots of semicolons.
1881
	 *   Most sites have zero Special Blog Tokens. Special tokens are only found in the
1882
	 *   JETPACK_BLOG_TOKEN constant.
1883
	 *
1884
	 * In particular, note that Normal Blog Tokens never start with ";" and that
1885
	 * Special Blog Tokens always do.
1886
	 *
1887
	 * When searching for a matching Blog Tokens, Blog Tokens are examined in the following
1888
	 * order:
1889
	 * 1. Defined Special Blog Tokens (via the JETPACK_BLOG_TOKEN constant)
1890
	 * 2. Stored Normal Tokens (via Jetpack_Options::get_option( 'blog_token' ))
1891
	 * 3. Defined Normal Tokens (via the JETPACK_BLOG_TOKEN constant)
1892
	 *
1893
	 * @param int|false    $user_id   false: Return the Blog Token. int: Return that user's User Token.
1894
	 * @param string|false $token_key If provided, check that the token matches the provided input.
1895
	 * @param bool|true    $suppress_errors If true, return a falsy value when the token isn't found; When false, return a descriptive WP_Error when the token isn't found.
1896
	 *
1897
	 * @return object|false
1898
	 */
1899
	public function get_access_token( $user_id = false, $token_key = false, $suppress_errors = true ) {
1900
		$possible_special_tokens = array();
1901
		$possible_normal_tokens  = array();
1902
		$user_tokens             = \Jetpack_Options::get_option( 'user_tokens' );
1903
1904
		if ( $user_id ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $user_id of type false|integer is loosely compared to true; this is ambiguous if the integer can be zero. You might want to explicitly use !== null instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For integer values, zero is a special case, in particular the following results might be unexpected:

0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
Loading history...
1905
			if ( ! $user_tokens ) {
1906
				return $suppress_errors ? false : new \WP_Error( 'no_user_tokens' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_user_tokens'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1907
			}
1908
			if ( self::JETPACK_MASTER_USER === $user_id ) {
1909
				$user_id = \Jetpack_Options::get_option( 'master_user' );
1910
				if ( ! $user_id ) {
1911
					return $suppress_errors ? false : new \WP_Error( 'empty_master_user_option' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'empty_master_user_option'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1912
				}
1913
			}
1914
			if ( ! isset( $user_tokens[ $user_id ] ) || ! $user_tokens[ $user_id ] ) {
1915
				return $suppress_errors ? false : new \WP_Error( 'no_token_for_user', sprintf( 'No token for user %d', $user_id ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_token_for_user'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1916
			}
1917
			$user_token_chunks = explode( '.', $user_tokens[ $user_id ] );
1918 View Code Duplication
			if ( empty( $user_token_chunks[1] ) || empty( $user_token_chunks[2] ) ) {
1919
				return $suppress_errors ? false : new \WP_Error( 'token_malformed', sprintf( 'Token for user %d is malformed', $user_id ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'token_malformed'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1920
			}
1921 View Code Duplication
			if ( $user_token_chunks[2] !== (string) $user_id ) {
1922
				return $suppress_errors ? false : new \WP_Error( 'user_id_mismatch', sprintf( 'Requesting user_id %d does not match token user_id %d', $user_id, $user_token_chunks[2] ) );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'user_id_mismatch'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1923
			}
1924
			$possible_normal_tokens[] = "{$user_token_chunks[0]}.{$user_token_chunks[1]}";
1925
		} else {
1926
			$stored_blog_token = \Jetpack_Options::get_option( 'blog_token' );
1927
			if ( $stored_blog_token ) {
1928
				$possible_normal_tokens[] = $stored_blog_token;
1929
			}
1930
1931
			$defined_tokens_string = Constants::get_constant( 'JETPACK_BLOG_TOKEN' );
1932
1933
			if ( $defined_tokens_string ) {
1934
				$defined_tokens = explode( ',', $defined_tokens_string );
1935
				foreach ( $defined_tokens as $defined_token ) {
1936
					if ( ';' === $defined_token[0] ) {
1937
						$possible_special_tokens[] = $defined_token;
1938
					} else {
1939
						$possible_normal_tokens[] = $defined_token;
1940
					}
1941
				}
1942
			}
1943
		}
1944
1945
		if ( self::MAGIC_NORMAL_TOKEN_KEY === $token_key ) {
1946
			$possible_tokens = $possible_normal_tokens;
1947
		} else {
1948
			$possible_tokens = array_merge( $possible_special_tokens, $possible_normal_tokens );
1949
		}
1950
1951
		if ( ! $possible_tokens ) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $possible_tokens of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
1952
			return $suppress_errors ? false : new \WP_Error( 'no_possible_tokens' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_possible_tokens'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1953
		}
1954
1955
		$valid_token = false;
1956
1957
		if ( false === $token_key ) {
1958
			// Use first token.
1959
			$valid_token = $possible_tokens[0];
1960
		} elseif ( self::MAGIC_NORMAL_TOKEN_KEY === $token_key ) {
1961
			// Use first normal token.
1962
			$valid_token = $possible_tokens[0]; // $possible_tokens only contains normal tokens because of earlier check.
1963
		} else {
1964
			// Use the token matching $token_key or false if none.
1965
			// Ensure we check the full key.
1966
			$token_check = rtrim( $token_key, '.' ) . '.';
1967
1968
			foreach ( $possible_tokens as $possible_token ) {
1969
				if ( hash_equals( substr( $possible_token, 0, strlen( $token_check ) ), $token_check ) ) {
1970
					$valid_token = $possible_token;
1971
					break;
1972
				}
1973
			}
1974
		}
1975
1976
		if ( ! $valid_token ) {
1977
			return $suppress_errors ? false : new \WP_Error( 'no_valid_token' );
0 ignored issues
show
Unused Code introduced by
The call to WP_Error::__construct() has too many arguments starting with 'no_valid_token'.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
1978
		}
1979
1980
		return (object) array(
1981
			'secret'           => $valid_token,
1982
			'external_user_id' => (int) $user_id,
1983
		);
1984
	}
1985
1986
	/**
1987
	 * In some setups, $HTTP_RAW_POST_DATA can be emptied during some IXR_Server paths
1988
	 * since it is passed by reference to various methods.
1989
	 * Capture it here so we can verify the signature later.
1990
	 *
1991
	 * @param Array $methods an array of available XMLRPC methods.
1992
	 * @return Array the same array, since this method doesn't add or remove anything.
1993
	 */
1994
	public function xmlrpc_methods( $methods ) {
1995
		$this->raw_post_data = $GLOBALS['HTTP_RAW_POST_DATA'];
1996
		return $methods;
1997
	}
1998
1999
	/**
2000
	 * Resets the raw post data parameter for testing purposes.
2001
	 */
2002
	public function reset_raw_post_data() {
2003
		$this->raw_post_data = null;
2004
	}
2005
2006
	/**
2007
	 * Registering an additional method.
2008
	 *
2009
	 * @param Array $methods an array of available XMLRPC methods.
2010
	 * @return Array the amended array in case the method is added.
2011
	 */
2012
	public function public_xmlrpc_methods( $methods ) {
2013
		if ( array_key_exists( 'wp.getOptions', $methods ) ) {
2014
			$methods['wp.getOptions'] = array( $this, 'jetpack_get_options' );
2015
		}
2016
		return $methods;
2017
	}
2018
2019
	/**
2020
	 * Handles a getOptions XMLRPC method call.
2021
	 *
2022
	 * @param Array $args method call arguments.
2023
	 * @return an amended XMLRPC server options array.
2024
	 */
2025
	public function jetpack_get_options( $args ) {
2026
		global $wp_xmlrpc_server;
2027
2028
		$wp_xmlrpc_server->escape( $args );
2029
2030
		$username = $args[1];
2031
		$password = $args[2];
2032
2033
		$user = $wp_xmlrpc_server->login( $username, $password );
2034
		if ( ! $user ) {
2035
			return $wp_xmlrpc_server->error;
2036
		}
2037
2038
		$options   = array();
2039
		$user_data = $this->get_connected_user_data();
2040
		if ( is_array( $user_data ) ) {
2041
			$options['jetpack_user_id']         = array(
2042
				'desc'     => __( 'The WP.com user ID of the connected user', 'jetpack' ),
2043
				'readonly' => true,
2044
				'value'    => $user_data['ID'],
2045
			);
2046
			$options['jetpack_user_login']      = array(
2047
				'desc'     => __( 'The WP.com username of the connected user', 'jetpack' ),
2048
				'readonly' => true,
2049
				'value'    => $user_data['login'],
2050
			);
2051
			$options['jetpack_user_email']      = array(
2052
				'desc'     => __( 'The WP.com user email of the connected user', 'jetpack' ),
2053
				'readonly' => true,
2054
				'value'    => $user_data['email'],
2055
			);
2056
			$options['jetpack_user_site_count'] = array(
2057
				'desc'     => __( 'The number of sites of the connected WP.com user', 'jetpack' ),
2058
				'readonly' => true,
2059
				'value'    => $user_data['site_count'],
2060
			);
2061
		}
2062
		$wp_xmlrpc_server->blog_options = array_merge( $wp_xmlrpc_server->blog_options, $options );
2063
		$args                           = stripslashes_deep( $args );
2064
		return $wp_xmlrpc_server->wp_getOptions( $args );
2065
	}
2066
2067
	/**
2068
	 * Adds Jetpack-specific options to the output of the XMLRPC options method.
2069
	 *
2070
	 * @param Array $options standard Core options.
2071
	 * @return Array amended options.
2072
	 */
2073
	public function xmlrpc_options( $options ) {
2074
		$jetpack_client_id = false;
2075
		if ( $this->is_active() ) {
2076
			$jetpack_client_id = \Jetpack_Options::get_option( 'id' );
2077
		}
2078
		$options['jetpack_version'] = array(
2079
			'desc'     => __( 'Jetpack Plugin Version', 'jetpack' ),
2080
			'readonly' => true,
2081
			'value'    => Constants::get_constant( 'JETPACK__VERSION' ),
2082
		);
2083
2084
		$options['jetpack_client_id'] = array(
2085
			'desc'     => __( 'The Client ID/WP.com Blog ID of this site', 'jetpack' ),
2086
			'readonly' => true,
2087
			'value'    => $jetpack_client_id,
2088
		);
2089
		return $options;
2090
	}
2091
2092
	/**
2093
	 * Resets the saved authentication state in between testing requests.
2094
	 */
2095
	public function reset_saved_auth_state() {
2096
		$this->xmlrpc_verification = null;
2097
	}
2098
2099
	/**
2100
	 * Sign a user role with the master access token.
2101
	 * If not specified, will default to the current user.
2102
	 *
2103
	 * @access public
2104
	 *
2105
	 * @param string $role    User role.
2106
	 * @param int    $user_id ID of the user.
0 ignored issues
show
Documentation introduced by
Should the type for parameter $user_id not be integer|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...
2107
	 * @return string Signed user role.
2108
	 */
2109
	public function sign_role( $role, $user_id = null ) {
2110
		if ( empty( $user_id ) ) {
2111
			$user_id = (int) get_current_user_id();
2112
		}
2113
2114
		if ( ! $user_id ) {
2115
			return false;
2116
		}
2117
2118
		$token = $this->get_access_token();
2119
		if ( ! $token || is_wp_error( $token ) ) {
2120
			return false;
2121
		}
2122
2123
		return $role . ':' . hash_hmac( 'md5', "{$role}|{$user_id}", $token->secret );
2124
	}
2125
}
2126