vechain-sdk-js icon indicating copy to clipboard operation
vechain-sdk-js copied to clipboard

Investigate how to also support axios or other http providers

Open victhorbi opened this issue 7 months ago • 1 comments

Context

Axios was originally chosen by the team to handle http requests. However, it was coming with an additional dependency, and it was causing conflicts for projects willing to migrate to using the sdk that were coming from thor-devkit. The Team eventually agreed to move to Fetch, that is native in javascript.

Description

Since Axios comes with many pros, like automatic polyfills, easy syntax and suitability for advanced use-cases, we are keen to support it either, as a pluggable part of the sdk. Best would be to support a generic http provider, it's acceptable to only support axios as the alternative.

Acceptance criteria

  • the custom provider must be specified with an optional argument
  • the use of the standard (fetch) vs the custom http provider must be transparent for the sdk
  • shall not figure as dependency

victhorbi avatar Jul 29 '24 09:07 victhorbi