🐛 Bootloader: Bail early on wp-login.php/wp-admin, and some other /wp- requests

[retlehs]

• Fix wp-login.php
• Fix wp-admin / admin-ajax.php requests
• Fix other requests? wp-comments-post.php, wp-signup.php?

Fix #337

[dsturm]

Thanks for starting a fix so quickly.

I also noticed that there were problems with REST queries in the block editor. But I'm not sure if it's just the cookie not being set.

[retlehs]

Pushed an update that should fix those I think? If not, do you mind showing which requests are failing for you still?

I've tested on:

• The roots.io site
• Radicle setup
• Stock Bedrock/Acorn/Sage setup

[dsturm]

@retlehs Worked for me! Thank you 💪

[QWp6t]

LGTM!

[dr5hn]

@QWp6t //cc @retlehs Doesn't work if we've hided login behind some custom path.

Can we pass a custom path using config?

[retlehs]

@dr5hn Does login_url() return the correct URL on your setup?

[dr5hn]

@retlehs Yes it return the hidden admin path url.

https://x***x.com/secret/

When i make this change, login works!

diff --git a/src/Roots/Acorn/Bootloader.php b/src/Roots/Acorn/Bootloader.php
index fc0e8c9..b57adfc 100644
--- a/src/Roots/Acorn/Bootloader.php
+++ b/src/Roots/Acorn/Bootloader.php
@@ -266,6 +266,7 @@ class Bootloader
             '/wp-login.php',
             '/wp-signup.php',
             '/wp-admin/',
+            '/secret/'
         ])) {
             return; // Let WordPress handle these requests
         }

Adding this dynamic path will get this working

parse_url(wp_login_url(), PHP_URL_PATH),