Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
11/03/2024 06:19:06 PM (5 months ago)
Author:
espellcaste
Message:

A user is no longer de-authenticated when making REST API requests.

We are introducing a new BP_LoggedIn_User class to fetch data about a BuddyPress logged-in user. This new addition fixes an issue where a user could be de-authenticated when making REST API requests.

Props dcavins, DJPaul, johnjamesjacoby, and imath.

Closes ​https://github.com/buddypress/buddypress/pull/395
See #9229 and #9145
Fixes #7658

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/testcases/groups/test-group-membership-request-controller.php

    r14026 r14070  
    8484        );
    8585
    86         $this->bp::set_current_user( $this->user );
     86        wp_set_current_user( $this->user );
    8787
    8888        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    … …  
    162162        );
    163163
    164         $this->bp::set_current_user( $this->g1admin );
     164        wp_set_current_user( $this->g1admin );
    165165
    166166        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    … …  
    192192        );
    193193
    194         $this->bp::set_current_user( $u );
     194        wp_set_current_user( $u );
    195195
    196196        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    … …  
    213213     */
    214214    public function test_get_items_user_is_not_logged_in() {
    215         $this->bp::set_current_user( 0 );
     215        wp_set_current_user( 0 );
    216216
    217217        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    … …  
    241241        );
    242242
    243         $this->bp::set_current_user( $u2 );
     243        wp_set_current_user( $u2 );
    244244        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    245245        $request->set_query_params(
    … …  
    268268        );
    269269
    270         $this->bp::set_current_user( $u2 );
     270        wp_set_current_user( $u2 );
    271271        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    272272        $request->set_query_params(
    … …  
    285285     */
    286286    public function test_get_items_invalid_group() {
    287         $this->bp::set_current_user( $this->user );
     287        wp_set_current_user( $this->user );
    288288
    289289        $request = new WP_REST_Request( 'GET', $this->endpoint_url );
    … …  
    312312        );
    313313
    314         $this->bp::set_current_user( $this->user );
     314        wp_set_current_user( $this->user );
    315315
    316316        $request = new WP_REST_Request( 'GET', $this->endpoint_url . '/' . $request_id );
    … …  
    361361        );
    362362
    363         $this->bp::set_current_user( 0 );
     363        wp_set_current_user( 0 );
    364364        $request = new WP_REST_Request( 'GET', $this->endpoint_url . '/' . $request_id );
    365365        $request->set_param( 'context', 'view' );
    … …  
    373373     */
    374374    public function test_get_item_invalid_membership_request() {
    375         $this->bp::set_current_user( $this->user );
     375        wp_set_current_user( $this->user );
    376376
    377377        $request = new WP_REST_Request( 'GET', $this->endpoint_url . '/' . REST_TESTS_IMPOSSIBLY_HIGH_NUMBER );
    … …  
    396396        );
    397397
    398         $this->bp::set_current_user( $u2 );
     398        wp_set_current_user( $u2 );
    399399
    400400        $request = new WP_REST_Request( 'GET', $this->endpoint_url . '/' . $request_id );
    … …  
    410410    public function test_create_item() {
    411411        $u = static::factory()->user->create();
    412         $this->bp::set_current_user( $this->user );
     412        wp_set_current_user( $this->user );
    413413
    414414        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    436436        $u = static::factory()->user->create();
    437437
    438         $this->bp::set_current_user( $u );
     438        wp_set_current_user( $u );
    439439
    440440        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    462462        $u = static::factory()->user->create();
    463463
    464         $this->bp::set_current_user( 0 );
     464        wp_set_current_user( 0 );
    465465
    466466        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    481481     */
    482482    public function test_create_item_invalid_member() {
    483         $this->bp::set_current_user( $this->user );
     483        wp_set_current_user( $this->user );
    484484
    485485        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    500500     */
    501501    public function test_create_item_invalid_group() {
    502         $this->bp::set_current_user( $this->user );
     502        wp_set_current_user( $this->user );
    503503
    504504        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    522522
    523523        $this->bp::add_user_to_group( $u, $this->group_id );
    524         $this->bp::set_current_user( $u );
     524        wp_set_current_user( $u );
    525525
    526526        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    551551        );
    552552
    553         $this->bp::set_current_user( $u );
     553        wp_set_current_user( $u );
    554554
    555555        $request = new WP_REST_Request( 'POST', $this->endpoint_url );
    … …  
    571571    public function test_update_item() {
    572572        $u = static::factory()->user->create();
    573         $this->bp::set_current_user( $this->user );
     573        wp_set_current_user( $this->user );
    574574
    575575        $request_id = groups_send_membership_request(
    … …  
    598598    public function test_update_item_as_group_admin() {
    599599        $u = static::factory()->user->create();
    600         $this->bp::set_current_user( $u );
     600        wp_set_current_user( $u );
    601601        $request_id = groups_send_membership_request(
    602602            array(
    … …  
    606606        );
    607607
    608         $this->bp::set_current_user( $this->g1admin );
     608        wp_set_current_user( $this->g1admin );
    609609        $request = new WP_REST_Request( 'PUT', $this->endpoint_url . '/' . $request_id );
    610610        $request->set_param( 'context', 'edit' );
    … …  
    633633        );
    634634
    635         $this->bp::set_current_user( 0 );
     635        wp_set_current_user( 0 );
    636636        $request = new WP_REST_Request( 'PUT', $this->endpoint_url . '/' . $request_id );
    637637        $request->set_param( 'context', 'edit' );
    … …  
    655655        );
    656656
    657         $this->bp::set_current_user( $u2 );
     657        wp_set_current_user( $u2 );
    658658
    659659        $request = new WP_REST_Request( 'PUT', $this->endpoint_url . '/' . $request_id );
    … …  
    668668     */
    669669    public function test_update_item_invalid_id() {
    670         $this->bp::set_current_user( $this->user );
     670        wp_set_current_user( $this->user );
    671671
    672672        $request = new WP_REST_Request( 'PUT', $this->endpoint_url . '/' . REST_TESTS_IMPOSSIBLY_HIGH_NUMBER );
    … …  
    682682    public function test_delete_item() {
    683683        $u = static::factory()->user->create();
    684         $this->bp::set_current_user( $this->user );
     684        wp_set_current_user( $this->user );
    685685
    686686        $request_id = groups_send_membership_request(
    … …  
    707707    public function test_delete_item_as_requestor() {
    708708        $u = static::factory()->user->create();
    709         $this->bp::set_current_user( $u );
     709        wp_set_current_user( $u );
    710710
    711711        $request_id = groups_send_membership_request(
    … …  
    732732    public function test_delete_item_as_group_admin() {
    733733        $u = static::factory()->user->create();
    734         $this->bp::set_current_user( $u );
     734        wp_set_current_user( $u );
    735735        $request_id = groups_send_membership_request(
    736736            array(
    … …  
    740740        );
    741741
    742         $this->bp::set_current_user( $this->g1admin );
     742        wp_set_current_user( $this->g1admin );
    743743        $request = new WP_REST_Request( 'DELETE', $this->endpoint_url . '/' . $request_id );
    744744        $request->set_param( 'context', 'edit' );
    … …  
    765765        );
    766766
    767         $this->bp::set_current_user( 0 );
     767        wp_set_current_user( 0 );
    768768        $request = new WP_REST_Request( 'DELETE', $this->endpoint_url . '/' . $request_id );
    769769        $request->set_param( 'context', 'edit' );
    … …  
    787787        );
    788788
    789         $this->bp::set_current_user( $u2 );
     789        wp_set_current_user( $u2 );
    790790
    791791        $request = new WP_REST_Request( 'DELETE', $this->endpoint_url . '/' . $request_id );
Note: See TracChangeset for help on using the changeset viewer.