Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: Remove not used blocking http client #1895

Merged
merged 3 commits into from
Apr 10, 2023
Merged

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Apr 10, 2023

We will not relay on blocking http client for signing.

Signed-off-by: Xuanwo <github@xuanwo.io>
Xuanwo added 2 commits April 11, 2023 00:58
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo
Copy link
Member Author

Xuanwo commented Apr 10, 2023

cc @ClSlaid, please review again.

@Xuanwo Xuanwo merged commit 64f0e9f into main Apr 10, 2023
@Xuanwo Xuanwo deleted the remove_blocking_client branch April 10, 2023 17:07
wcy-fdu pushed a commit to wcy-fdu/opendal that referenced this pull request Apr 12, 2023
* refactor: Remove not used blocking http client

Signed-off-by: Xuanwo <github@xuanwo.io>

* Polish naming

Signed-off-by: Xuanwo <github@xuanwo.io>

* Fix API

Signed-off-by: Xuanwo <github@xuanwo.io>

---------

Signed-off-by: Xuanwo <github@xuanwo.io>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants