Changeset 10086 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/clients/class-github.php

Timestamp:

07/21/2020 04:02:38 AM (5 years ago)

Author:

dd32

Message:

Plugin Directory: Cache the expiry of the GitHub transient in the transient to ensure a stale transient isn't used on long-running requests.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/clients/class-github.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/clients/class-github.php

@@ -43 +43 @@
      */
     protected static function get_app_install_token() {
-        $token = false; get_site_transient( __CLASS__ . 'app_install_token' );
-        if ( $token ) {
-            return $token;
+        $token = get_site_transient( __CLASS__ . 'app_install_token' );
+        if ( $token && is_array( $token ) && $token['exp'] > time() ) {
+            return $token['token'];
         }
 
@@ -88 +88 @@
 
         // Cache the token for 1 minute less than what it's valid for.
-        set_site_transient( __CLASS__ . 'app_install_token', $token, $token_exp - time() - MINUTE_IN_SECONDS );
+        $exp = $token_exp - time() - MINUTE_IN_SECONDS;
+        set_site_transient( __CLASS__ . 'app_install_token', [ 'exp' => time() + $exp, 'token' => $token ], $exp );
 
         return $token;
@@ -98 +99 @@
     protected static function get_jwt_app_token() {
         $token = get_site_transient( __CLASS__ . 'app_token' );
-        if ( $token ) {
-            return $token;
+        if ( $token && is_array( $token ) && $token['exp'] > time() ) {
+            return $token['token'];
         }
 
@@ -119 +120 @@
 
         // Cache it for 9 mins (It's valid for 10min).
-        set_site_transient( __CLASS__ . 'app_token', $token, 9 * MINUTE_IN_SECONDS );
+        $exp = 9 * MINUTE_IN_SECONDS;
+        set_site_transient( __CLASS__ . 'app_token', [ 'exp' => time() + $exp, 'token' => $token ], $exp );
 
         return $token;