Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#7860 closed defect (bug) (fixed)

502 errors when using the BP Login Widget

Reported by: r-a-y's profile r-a-y Owned by: r-a-y's profile r-a-y
Milestone: 3.1.0 Priority: normal
Severity: normal Version: 3.0.0
Component: Core Keywords: has-patch
Cc:

Description

There are various reports about users encountering 502 errors with the login widget.

We made changes in #7522 to use wp_login_url() instead of site_url( 'wp-login.php' ).

But, I think we should revert it for the login widget since WP's own wp_login_form() function uses site_url( 'wp-login.php', 'login_post' ).

Attachments (1)

7860.01.patch​ (825 bytes) - added by r-a-y 7 years ago.

Download all attachments as: .zip

Change History (3)

@r-a-y
7 years ago

#1 @r-a-y
7 years ago

  • Owner set to r-a-y
  • Resolution set to fixed
  • Status changed from new to closed

In 12117:

Widgets: Revert wp_login_url() change for the Login Widget.

In #7522, we switched from using site_url( 'wp-login.php' ) to
wp_login_url() for the form action. But, wp_login_url() doesn't take
into account HTTPS post submissions. So this commit reverts this change.

Props eric01.

Fixes #7860 (3.x branch).

#2 @r-a-y
7 years ago

In 12118:

Widgets: Revert wp_login_url() change for the Login Widget.

In #7522, we switched from using site_url( 'wp-login.php' ) to
wp_login_url() for the form action. But, wp_login_url() doesn't take
into account HTTPS post submissions. So this commit reverts this change.

Props eric01.

Fixes #7860 (trunk).

Note: See TracTickets for help on using tickets.