[wp-meta] [Making WordPress.org] #5061: Upgrade GitHub OAuth Usage

Making WordPress.org noreply at wordpress.org
Thu Feb 27 03:19:06 UTC 2020


#5061: Upgrade GitHub OAuth Usage
----------------------+-----------------------
 Reporter:  pento     |       Owner:  dd32
     Type:  task      |      Status:  accepted
 Priority:  normal    |   Milestone:
Component:  Profiles  |  Resolution:
 Keywords:            |
----------------------+-----------------------

Comment (by dd32):

 I think this is fixed as of r15784-dotorg.

 {{{
 #!diff
 Index: class-api-endpoint.php
 ===================================================================
 --- class-api-endpoint.php      (revision 15777)
 +++ class-api-endpoint.php      (working copy)
 @@ -47,8 +47,8 @@

                 // Revoke our access to that users token
                 $api = $this->make_api_request(
 -                       '/applications/' . CLIENT_ID . '/grants/' .
 $access_token,
 -                       null,
 +                       '/applications/' . CLIENT_ID . '/grant',
 +                       json_encode( [ 'access_token' => $access_token ]
 ),
                         [ 'Authorization' => 'BASIC ' . base64_encode(
 CLIENT_ID . ':' . CLIENT_SECRET ) ],
                         'DELETE'
                 );
 }}}

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/5061#comment:3>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list