Skip to content
This repository has been archived by the owner on Feb 10, 2025. It is now read-only.

Async HTTP Support #121

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Async HTTP Support #121

wants to merge 2 commits into from

Conversation

Roman505050
Copy link

The synchronous code has been rewritten to be asynchronous, providing users with greater flexibility and efficiency. Asynchronous programming is a crucial aspect of modern applications, enabling concurrent operations and enhancing overall project performance.

To achieve this, the asyncio package was utilized to rewrite the KucoinBaseRestApi class asynchronously using the aiohttp library.

@ISAAC-XXYYZZ
Copy link
Contributor

Impressive work! however, I cannot accept your request at the moment as I am unable to properly verify this change. meanwhile, this is indeed a poor design. We are working on an updated version of the API that will be more user-friendly and offer better abstraction capabilities, with built-in error handling and robust HTTP management. So, please stay tuned!

@Graeme22
Copy link

May I suggest httpx? It is very similar to requests, and supports both sync and async out-of-the-box.

@ISAAC-XXYYZZ
Copy link
Contributor

It's been a while since our last conversation, and some things have changed. We've released a new Python SDK, which is a complete rewrite of all implementations. You can check it out here: KuCoin Universal SDK - Python.

As a result, this repository is no longer maintained and will not accept new PRs. We encourage community users to migrate to the new SDK, which we will continue to iterate on.

We're also working on expanding support for multiple languages, and while asynchronous support is in our plans, it won't be implemented immediately.

Community contributions to the new SDK are highly welcome, and we will prioritize merging PRs as soon as possible.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants