Changeset 36867

Timestamp:

03/06/2016 03:05:46 AM (9 years ago)

Author:

jeremyfelt

Message:

Multisite: Handle redirect to a user's subdomain properly during login

wp-login.php uses wp_safe_redirect() for all redirects, even those that do not involve unsafe data from the request or referer.

When a user of a subdomain site attempts to login to a network site they do not have access to, the host in the redirect URL is treated as unsafe by wp_safe_redirect() as it has no immediate awareness as to which hosts are valid on the network. On a subdirectoy network, everything works as expected because the host is the same.

In this specific block of wp-login.php, all URLs are generated by WordPress and we can use wp_redirect() to handle the redirects. Users authenticating via other network sites will now be redirected properly. Hosts passed via the redirect_to query var will continue to be handled by wp_safe_redirect().

Fixes #30598.

File:

• trunk/src/wp-login.php (modified) (1 diff)

trunk/src/wp-login.php

@@ -831 +831 @@
             elseif ( !$user->has_cap('edit_posts') )
                 $redirect_to = $user->has_cap( 'read' ) ? admin_url( 'profile.php' ) : home_url();
+
+            wp_redirect( $redirect_to );
+            exit();
         }
         wp_safe_redirect($redirect_to);