fix: proxy url and fallback, logging

amplitude · Aug 7, 2024 · 4720f48 · 4720f48
1 parent 03ac301
commit 4720f48
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/main/kotlin/LocalEvaluationClient.kt b/src/main/kotlin/LocalEvaluationClient.kt
@@ -106,7 +106,6 @@ class LocalEvaluationClient internal constructor(
         ) {
             evaluation.evaluate(enrichedUser.toEvaluationContext(), sortedFlagConfigs)
         }
-        Logger.d("evaluate - user=$enrichedUser, result=$evaluationResults")
         val variants = evaluationResults.toVariants()
         assignmentService?.track(Assignment(user, variants))
         return variants

diff --git a/src/main/kotlin/cohort/CohortApi.kt b/src/main/kotlin/cohort/CohortApi.kt
@@ -70,6 +70,7 @@ internal class DynamicCohortApi(
             } catch (e: CohortTooLargeException) {
                 throw e
             } catch (e: Exception) {
+                Logger.e("Downloading cohort $cohortId from proxy failed. Falling back to Amplitude.", e)
                 metrics.onCohortDownloadOriginFallback(e)
                 getCohort(serverUrl, cohortId, cohort)
             }
@@ -92,7 +93,7 @@ internal class DynamicCohortApi(
                 queries["lastModified"] = "${cohort.lastModified}"
             }
             httpClient.get<GetCohortResponse>(
-                serverUrl = serverUrl,
+                serverUrl = url,
                 path = "sdk/v1/cohort/$cohortId",
                 headers = headers,
                 queries = queries,