LIT-Protocol / js-sdk

The Lit Protocol SDK provides developers with a framework for implementing Lit functionality into their own applications.
https://developer.litprotocol.com
MIT License
99 stars 59 forks source link

Fix/execute with retry wrapper exec #479

Closed joshLong145 closed 3 months ago

joshLong145 commented 4 months ago

Description

Removes executeWithRetry as we no longer need to retry operations due to network stability fixes. We now will only retry operations which are critical to the standup of the sdk. The included node endpoints which are eligible for retry at this time are:

Status codes we will retry on:

Introduces a new package fetch-retry which now handles retry logic for single requests. As we no longer need to group requests and retry as single network operation.

Type of change

How Has This Been Tested?

Tested with local testing to force endpoint failure to observe retry.

Checklist:

joshLong145 commented 4 months ago

Screenshot 2024-06-03 at 10 21 10 AM After doing some testing in browser environments it looks like the umd bundle target is non compatible with our build pipeline. Going to make a task in the current sprint to upgrade fetch-retry to a modern cross target bundling implementation. Once complete this pr can be merged.

joshLong145 commented 3 months ago

Closing for #497