[buddypress-trac] [BuddyPress Trac] #6528: bp_verify_nonce_request() not working correctly for those using a port in URL

[buddypress-trac]

#6528: bp_verify_nonce_request() not working correctly for those using a port in
URL
------------------------------+---------------------
 Reporter:  r-a-y             |       Owner:  r-a-y
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  2.4
Component:  Component - Core  |     Version:  1.6
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+---------------------
Changes (by r-a-y):

 * owner:   => r-a-y
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"10044"]:
 {{{
 #!CommitTicketReference repository="" revision="10044"
 Core: Fix issue with `bp_verify_nonce_request()` when a port is used in
 the home URL.

 When a site is using a port for their home URL,
 `bp_verify_nonce_request()`
 can sometimes fail.  This is because the port is duplicated twice due to
 `$_SERVER['HTTP_HOST']` including the port number and we're appending the
 port number again with `$_SERVER['SERVER_PORT']`.

 This commit appends `$_SERVER['SERVER_PORT']` if `$_SERVER['HTTP_HOST']`
 does not already include the port number and includes a unit test.

 Fixes #6528.
 }}}

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6528#comment:4>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac