feat: [FFM-6509]: Clear metrics after 2 failed attempts (#62)

harness · Jan 18, 2023 · ac6688f · ac6688f
1 parent a907c38
commit ac6688f
Show file tree

Hide file tree

Showing 11 changed files with 42 additions and 34 deletions.
diff --git a/README.md b/README.md
@@ -120,21 +120,21 @@ In case you want to import this library directly (without having to use npm or y
 
 ```html
 <script type="module">
-  import { initialize, Event } from 'https://unpkg.com/@harnessio/ff-javascript-client-sdk@1.5.0/dist/sdk.client.js'
+  import { initialize, Event } from 'https://unpkg.com/@harnessio/ff-javascript-client-sdk@1.6.0/dist/sdk.client.js'
 </script>
 ```
 
 If you need to support old browsers which don't support ES Module:
 
 ```html
-<script src="https://unpkg.com/@harnessio/ff-javascript-client-sdk@1.5.0/dist/sdk.client.js"></script>
+<script src="https://unpkg.com/@harnessio/ff-javascript-client-sdk@1.6.0/dist/sdk.client.js"></script>
 <script>
   var initialize = HarnessFFSDK.initialize
   var Event = HarnessFFSDK.Event
 </script>
 ```
 
-Remember to change the version `1.5.0` in the unpkg url accordingly.
+Remember to change the version `1.6.0` in the unpkg url accordingly.
 
 ## License
 

diff --git a/dist/sdk.cjs.js b/dist/sdk.cjs.js
diff --git a/dist/sdk.client-iife.js b/dist/sdk.client-iife.js
diff --git a/dist/sdk.client.js b/dist/sdk.client.js
diff --git a/dist/sdk.esm.js b/dist/sdk.esm.js
diff --git a/examples/preact/package-lock.json b/examples/preact/package-lock.json
diff --git a/examples/react-redux/package-lock.json b/examples/react-redux/package-lock.json
diff --git a/examples/react/package-lock.json b/examples/react/package-lock.json
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@harnessio/ff-javascript-client-sdk",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "author": "Harness",
   "license": "Apache-2.0",
   "main": "dist/sdk.cjs.js",

diff --git a/src/index.ts b/src/index.ts
@@ -15,7 +15,7 @@ import type {
 import { Event } from './types'
 import { defaultOptions, logError, MIN_EVENTS_SYNC_INTERVAL } from './utils'
 
-const SDK_VERSION = '1.5.0'
+const SDK_VERSION = '1.6.0'
 const METRICS_VALID_COUNT_INTERVAL = 500
 const fetch = globalThis.fetch
 const EventSource = EventSourcePolyfill
@@ -109,6 +109,8 @@ const initialize = (apiKey: string, target: Target, options?: Options): Result =
     return data.authToken
   }
 
+  let failedMetricsCallCount = 0
+
   const scheduleSendingMetrics = () => {
     if (metrics.length) {
       logDebug('Sending metrics...', { metrics, evaluations })
@@ -161,8 +163,13 @@ const initialize = (apiKey: string, target: Target, options?: Options): Result =
       })
         .then(() => {
           metrics = []
+          failedMetricsCallCount = 0
         })
         .catch(error => {
+          if (failedMetricsCallCount++) {
+            metrics = []
+            failedMetricsCallCount = 0
+          }
           logDebug(error)
         })
         .finally(() => {