Plugin Directory: Logging in from a plugin page redirects to the home page

[SergeyBiryukov]

1. If you're logged in, log out.
2. Go to a plugin page like ​https://wordpress.org/plugins/akismet/.
3. Click Log In in the Toolbar.
4. Upon login, you're redirected to ​https://wordpress.org/plugins/ instead of the plugin page.

[SergeyBiryukov]

Support forums apparently have a workaround for this, added in [4613]. We could probably add a similar workaround to the Plugin Directory, but it seems like something that should be fixed in the WPOrg_SSO class itself.

[Otto42]

There's nothing wrong with the SSO class. If it's being passed the wrong redirect_to link, then that's the problem.

[SergeyBiryukov]

Replying to Otto42:

There's nothing wrong with the SSO class. If it's being passed the wrong redirect_to link, then that's the problem.

Shouldn't it redirect to the previous page by default? The workaround in [4613] doesn't seem specific to the forums, is there a reason the SSO class can't parse $_SERVER['REQUEST_URI'] itself?

[Otto42]

Shouldn't it redirect to the previous page by default?

Well, yes, but in this case it's actually sending a redirect_to with just the /plugins/ URL. So it's still getting the wrong URL from the plugins directory.

I'll look into it.

[Otto42]

In 6207:

Plugins: Fix the redirect_to URL in the same way as the support forums. Fixes #3108

[SergeyBiryukov]

In 6801:

Plugin Directory: In ::fix_login_url():

• Only adjust the login URL for Single Sign-On (SSO) plugin if the plugin is active, use the normal redirect_to method otherwise.
• Use $_SERVER['HTTP_HOST'] instead of hardcoded wordpress.org domain.

Merges [4944] from support forums.
See #3108.

[obenland]

I just ran into that weird WPOrg_SSO while trying to find out why I'm not being redirected to my profile when using wp_login_url().

This is clearly an issue with WPOrg_SSO::login_url() being used as a callback to the login_url filter and not handling the passed parameters correctly.

Is WPOrg_SSO::login_url() used anywhere other than as that filter callback that prevents us from just fixing the parameters?

[obenland]

@SergeyBiryukov @Otto42 Would you mind testing my patch on your sandboxes and see if it needs adjustment?

[obenland]

@SergeyBiryukov @Otto42 I'd like to land this this week, do you think you'll have time to take my patch for a spin today or tomrrow?

[Otto42]

Sorry, missed the notification a couple weeks ago. I'll check it out shortly.

[dd32]

3108.diff​ was basically committed in [7689] through #3794.

Apparently I've been running a variant of it for quite possibly just as long as the patch has been here and not seen any issues.

Closing this as fixed based on the above.