search

Tinkoff / invest-openapi-csharp-sdk

Apache License 2.0
100 stars 33 forks source link

Не хватает CancellationToken в методе SendStreamingRequestAsync #80

Closed danclax closed 3 years ago

danclax commented 3 years ago

Когда нужно остановить сервис, через cancellation token удобно (и это best practice) мягко и корректно завершать все асинхронные операции

danclax commented 3 years ago

Вот пример изменения кода - https://github.com/danclax/invest-openapi-csharp-sdk/pull/1 Интерфейс не ломает, зато дает возможность отменить процесс подключения

rus-art commented 3 years ago

Спасибо! Вы могли бы создать PR тут? Я что-то не вижу там ваш репозиторий в списке форков

danclax commented 3 years ago

Я как-то криво форканул. Сделал в новом репо и создал PR: https://github.com/TinkoffCreditSystems/invest-openapi-csharp-sdk/pull/84

rus-art commented 3 years ago

Спасибо!