fix bug in uploadAemAsset when used through browser

pk28yadav commented 11 months ago

Description

The change done in https://github.com/adobe/acc-js-sdk/pull/91 didn't work correctly when sdk is used inside browser. Noticed that we have two different implementations of transport layer - axios(node) & fetch(browser). I didn't knew that earlier & had only tested with a nodejs app. Two issues -

Axios automatically serialized json body but fetch doesn't.
Axios handled to return json response while fetch handled for string response.

Related Issue

ttps://github.com/adobe/acc-js-sdk/pull/91

Motivation and Context

How Has This Been Tested?

same as linked PR

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

pk28yadav commented 11 months ago

@mkiki , sorry for inconvenience, but my last code failed on browser. Please add review.

mkiki commented 11 months ago

Will fix the transport function to have a consistent behavior in future PR

adobe / acc-js-sdk