Merge pull request #371 from WPUserManager/#359-Prevent-site-access-t…

…o-visitors-setting-should-not-prevent-the-WP-REST-API-from-working #359 Prevent site access to visitors setting should not prevent the WP REST API from working
WPUserManager · Nov 18, 2023 · daee2e1 · daee2e1
2 parents 38d32d1 + d95d1b6
commit daee2e1
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/includes/actions.php b/includes/actions.php
@@ -412,6 +412,10 @@ function wpum_prevent_entire_site() {
 		return;
 	}
 
+	if ( wpum_is_rest_api_request() ) {
+		return;
+	}
+
 	if ( defined( 'WP_CLI' ) && WP_CLI ) {
 		return;
 	}

diff --git a/includes/functions.php b/includes/functions.php
@@ -1383,3 +1383,18 @@ function wpum_strip_slashes( $content ) {
 
 	return $content;
 }
+
+/**
+ * Returns true if the request is a REST API request.
+ *
+ * @return bool
+ */
+function wpum_is_rest_api_request() {
+	if ( empty( $_SERVER['REQUEST_URI'] ) ) {
+		return false;
+	}
+
+	$rest_prefix = trailingslashit( rest_get_url_prefix() );
+
+	return ( false !== strpos( $_SERVER['REQUEST_URI'], $rest_prefix ) ); // phpcs:ignore
+}
-Original file line number
+Diff line change
@@ Expand Up / @@ -412,6 +412,10 @@ function wpum_prevent_entire_site() { @@
     		return;
     	}
+    	if ( wpum_is_rest_api_request() ) {
+    		return;
+    	}
     	if ( defined( 'WP_CLI' ) && WP_CLI ) {
     		return;
     	}
@@ Expand Down @@