alipay-sdk-nodejs-all icon indicating copy to clipboard operation
alipay-sdk-nodejs-all copied to clipboard

Published 20 hours ago • verified publisher icon alipay

Replaces deprecated request dependency

Open perrin4869 opened this issue 4 years ago • 1 comments

I tried to leave everything backwards compatible. All the tests that were passing are still passing now. Since urllib is not required anymore because bent can do everything urllib can, I went ahead and removed it too. It should make this library much lighter for the end user, especially since urllib was a massive dependency, but here was only used to make simple http requests. This should also enable bundling this library with rollup, since it no longer depends on libraries that hold circular dependencies.

perrin4869 avatar Aug 16 '20 23:08 perrin4869

@ottomao I resolved the conflicts and fixed the tests! Would be nice if you could review this, thanks!

perrin4869 avatar May 13 '22 11:05 perrin4869

Thanks a lot for your contribution and sorry for this late reply. We decide to use urllib as our http request library, because bent seems to be no longer maintained, and urllib is under our maintenance. The request dependency will be removed in #95 .

troyeagle avatar May 04 '23 10:05 troyeagle