Skip to content
Snippets Groups Projects
Verified Commit 1d7e609a authored by Andrei Mondoc's avatar Andrei Mondoc Committed by Kevin Cristiano
Browse files

add civi_wp_rest/controller/rest/permissions_check filter

parent 0c5336dd
No related branches found
No related tags found
No related merge requests found
...@@ -45,13 +45,52 @@ class Rest extends Base { ...@@ -45,13 +45,52 @@ class Rest extends Base {
*/ */
public function permissions_check( $request ) { public function permissions_check( $request ) {
if ( ! $this->is_valid_api_key( $request ) ) /**
return $this->civi_rest_error( __( 'Param api_key is not valid.', 'civicrm' ) ); * Opportunity to bypass CiviCRM's
* authentication ('api_key' and 'site_key'),
* return 'true' or 'false' to grant
* or deny access to this endpoint.
*
* To deny and throw an error, return either
* a string, an array, or a \WP_Error.
*
* NOTE: if you use your won authentication,
* you still must log in the user in order
* to respect/apply CiviCRM ACLs.
*
* @since 0.1
* @param null|bool|string|array|\WP_Error $grant_auth Grant, deny, or error
* @param \WP_REST_Request $request The request
*/
$grant_auth = apply_filters( 'civi_wp_rest/controller/rest/permissions_check', null, $request );
if ( ! $this->is_valid_site_key() ) if ( is_bool( $grant_auth ) ) {
return $this->civi_rest_error( __( 'Param key is not valid.', 'civicrm' ) );
return true; return $grant_auth;
} elseif ( is_string( $grant_auth ) ) {
return $this->civi_rest_error( $grant_auth );
} elseif ( is_array( $grant_auth ) ) {
return $this->civi_rest_error( __( 'CiviCRM WP REST permission check error.', 'civicrm' ), $grant_auth );
} elseif ( $grant_auth instanceof \WP_Error ) {
return $grant_auth;
} else {
if ( ! $this->is_valid_api_key( $request ) )
return $this->civi_rest_error( __( 'Param api_key is not valid.', 'civicrm' ) );
if ( ! $this->is_valid_site_key() )
return $this->civi_rest_error( __( 'Param key is not valid.', 'civicrm' ) );
return true;
}
} }
...@@ -358,7 +397,7 @@ class Rest extends Base { ...@@ -358,7 +397,7 @@ class Rest extends Base {
return [ return [
'key' => [ 'key' => [
'type' => 'string', 'type' => 'string',
'required' => true, 'required' => false,
'validate_callback' => function( $value, $request, $key ) { 'validate_callback' => function( $value, $request, $key ) {
return $this->is_valid_site_key(); return $this->is_valid_site_key();
...@@ -367,7 +406,7 @@ class Rest extends Base { ...@@ -367,7 +406,7 @@ class Rest extends Base {
], ],
'api_key' => [ 'api_key' => [
'type' => 'string', 'type' => 'string',
'required' => true, 'required' => false,
'validate_callback' => function( $value, $request, $key ) { 'validate_callback' => function( $value, $request, $key ) {
return $this->is_valid_api_key( $request ); return $this->is_valid_api_key( $request );
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment