search

galimru / tinkoff-invest-api

🚀 Tinkoff Invest client written on Java
Apache License 2.0
5 stars 2 forks source link

Ошибка при получении исторических свечей #1

Closed 60M6ER closed 3 years ago

60M6ER commented 3 years ago

Наткнулся на ошибку при получении исторический свечей. QueryConverterFactory нужно поменять шаблон даты. hh на HH. Тинькофф принимает в 24 часовом формате. Спасибо за клиент. сильно помог. СДК от тинькоффа багованное(

galimru commented 3 years ago

Рад слышать что клиент кому-то помогает 😌.

Спасибо за репорт, время поправлю в следующем минорном релизе v1.1.4

Хочу пердупредить, что клиент написал пару дней назад и в продакшене еще не использовал. Поэтому буду рад любым фидбэкам и баг репортам.

60M6ER commented 3 years ago

Я пока на вчерашней 1.0.0 Из проблем было вот это получение свечей. И то что ордербук приходит без детализации по аск/бид. Сегодня стрим у Вас появился. Вот думаю начать пробовать его. А то у тинькоффа стримы валятся периодически. А у Вас в стримах на одну подписку одно соединение получается?

galimru commented 3 years ago

Да, стримы только сегодня доделал. На одном соединение должно работать несколько подписок, но я не проверял этот кейс.

И пока не тестировал как оно себе ведет в случае плохого соединения. Подозреваю что надо добавить возможность реконнекта и повторной подписки на события.

galimru commented 3 years ago

Несколько подписок на одном соединении работает. Добавил тест https://github.com/galimru/tinkoff-invest-api/blob/96361e405da111fc06e6dffd89b1112fcc3cc36a/src/test/java/com/github/galimru/tinkoff/StreamingServiceTest.java#L92