New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

`REST_REQUEST` is defined after `plugins_loaded`. #11

Open

peterwilsoncc opened this issue Jun 28, 2019 · 0 comments

peterwilsoncc commented Jun 28, 2019 •

edited

Loading

The plugin runs the following on the plugins_loaded hook to determine if the site is making a rest request.

// Don't redirect REST API requests
if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
	return;
}

Unfortunately, REST_REQUEST is only defined after the plugins have loaded so after the code above is passed.

Moving the redirect to the send_headers action works around this.

The text was updated successfully, but these errors were encountered:

r-a-y added a commit to r-a-y/Mercator that referenced this issue


          Load 'redirect' module on 'send_headers' instead of 'plugins_loaded'.

c88e750

Fixes an issue with checking for REST API requests since
'plugins_loaded' runs before the REST API is loaded.

See humanmade/Mercator-Redirect#11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment