Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Переезд на ktor #1

Open
Anton3 opened this issue Nov 30, 2018 · 3 comments
Open

Переезд на ktor #1

Anton3 opened this issue Nov 30, 2018 · 3 comments

Comments

@Anton3
Copy link

Anton3 commented Nov 30, 2018

Добрый день!
Хочу переехать на v5.87 с помощью вашего нагенеренного API. Но сейчас я использую ktor, и переезжать на вашу httpclient-либу очень бы не хотелось.
Попробую как-нибудь прибить ktor сбоку гвоздями 😄

@alatushkin
Copy link
Owner

alatushkin commented Nov 30, 2018

@Anton3, Привет.
Переезд на ктор значится первым пунктом в списке целей).

https://github.com/alatushkin/common-http-client/blob/master/src/main/kotlin/name/alatushkin/httpclient/HttpClient.kt
Там интерфейс простой как палка.

Если уже работал с ktor - проблемы не составит реализовать и использовать его.

Если будешь реализовывать - сделай pls pr в репозиторий common-http-client с ктор-реализацией, добавим туда опциональную зависимость, проверку наличия и будем пользоваться им

@Anton3
Copy link
Author

Anton3 commented Nov 30, 2018

Тут надо переписывать код, который участвует в формировании HTTP-запросов, например, MethodExecutor. Одна из важных частей Ktor, если уж мы хотим под него переписываться — как раз builder запросов.

@alatushkin
Copy link
Owner

Неа. В этом и смысл введения этого интерфейса - отвязать от деталей работы с конкретной библиотекой. Инкапсуляция, все дела.
Нужно написать класс который бы реализовывал работу этого метода в терминах ктор (из сформировать запрос,выполнить его, транслировать ответ)

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

No branches or pull requests

2 participants