@audius/sdk 1.0.34 → 1.0.35

package/dist/services/creatorNode/CreatorNode.d.ts

@@ -76,7 +76,7 @@ export declare class CreatorNode {
      * @param timeout max time alloted for clock request
      * @param params optional query string params
      */
-    static getClockValue(endpoint: string, wallet: string, timeout: number, params?: Record<string, string>): Promise<any>;
+    static getClockValue(endpoint: string, wallet: string, timeout?: number, params?: Record<string, string>): Promise<any>;
     /**
      * Checks if a download is available from provided creator node endpoints
      * @param endpoints creator node endpoints

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@audius/sdk",
-  "version": "1.0.34",
+  "version": "1.0.35",
   "audius": {
-    "releaseSHA": "8f0ae0d181bf1ee2dcf5ccc26987aaf1d0861903"
+    "releaseSHA": "e40315da81c1fcf7d1d43d1b7a4bcefc5d1243d1"
   },
   "description": "",
   "main": "dist/index.cjs.js",

package/src/sanityChecks/rolloverNodes.ts

@@ -1,6 +1,7 @@
 import { Nullable, Utils } from '../utils'
 import { CreatorNode } from '../services/creatorNode'
 import type { AudiusLibs } from '../AudiusLibs'
+import maxBy from 'lodash/maxBy'
 
 const THREE_SECONDS = 3000
 const MAX_TRIES = 3
@@ -23,15 +24,26 @@ const checkPrimaryHealthy = async (
 }
 
 /** Gets new endpoints from a user's secondaries */
-const getNewPrimary = async (libs: AudiusLibs, secondaries: string[]) => {
-  for (const secondary of secondaries) {
-    const syncStatus = await libs.creatorNode?.getSyncStatus(secondary)
-    if (!syncStatus) continue
-    if (!syncStatus.isBehind) {
-      return secondary
-    }
+const getNewPrimary = async (secondaries: string[], wallet: string) => {
+  const secondaryStatuses = (
+    await Promise.all(
+      secondaries.map(async (secondary) => {
+        try {
+          const clockValue = await CreatorNode.getClockValue(secondary, wallet)
+          if (clockValue) return { secondary, clockValue }
+          return undefined
+        } catch (e) {
+          console.warn(e)
+          return undefined
+        }
+      })
+    )
+  ).filter(Boolean)
+  const max = maxBy(secondaryStatuses, (s) => s?.clockValue)?.secondary
+  if (!max) {
+    throw new Error(`Could not find valid secondaries for user ${secondaries}`)
   }
-  throw new Error(`Could not find valid secondaries for user ${secondaries}`)
+  return max
 }
 
 export const rolloverNodes = async (
@@ -54,7 +66,7 @@ export const rolloverNodes = async (
 
   try {
     // Get a new primary
-    const newPrimary = await getNewPrimary(libs, secondaries)
+    const newPrimary = await getNewPrimary(secondaries, user.wallet!)
     const index = secondaries.indexOf(newPrimary)
     // Get new secondaries and backfill up to 2
     let newSecondaries = [...secondaries]
@@ -85,7 +97,11 @@ export const rolloverNodes = async (
     console.debug(
       `Sanity Check - rolloverNodes - new nodes ${newMetadata.creator_node_endpoint}`
     )
-    await libs.User?.updateCreator(user.user_id, newMetadata)
+    await libs.User?.updateCreator(
+      user.user_id,
+      newMetadata,
+      true /* useEntityManager */
+    )
   } catch (e) {
     console.error(e)
   }

package/src/services/creatorNode/CreatorNode.ts

@@ -115,7 +115,7 @@ export class CreatorNode {
   static async getClockValue(
     endpoint: string,
     wallet: string,
-    timeout: number,
+    timeout?: number,
     params: Record<string, string> = {}
   ) {
     const baseReq: AxiosRequestConfig = {