Некорректное поведение NewClient

[asamats]

invest-api-go-sdk/investgo/client.go

Line 90 in b516087

if conf.AccountId == "" {

Метод открывает соединение, но неожиданно оказывается при открытии соединения у него есть и side-effect, открывет счет в песочнице. Не надо так. Этот метод должен только открывать соединение и все.

И никакой AccountId в конфиге не нужен. У пользователя 1+ аккаунтов.

[jstalex]

Здравствуйте! @asamats
Функция NewClient() создает экземпляр клиента для API, не просто открывает соединение.
Для работы с несколькими счетами логика заложена следующая:
один счет - один токен - один бот(приложение, использующее sdk)
То есть для торговли на нескольких счетах, нужно запускать несколько приложений. На наш взгляд это позволяет более гибко управлять счетами и доступами к ним у разных ботов.

[falkolab]

...Для работы с несколькими счетами логика заложена следующая: один счет - один токен - один бот(приложение, использующее sdk) То есть для торговли на нескольких счетах, нужно запускать несколько приложений. На наш взгляд это позволяет более гибко управлять счетами и доступами к ним у разных ботов.

А лимиты на каждый счет/токен тоже свои?

Согласен с @asamats не должно быть сайд эффекта, NewClient должно отдать ошибку не задан счет если он не указан. Не зависимо как вы подразумеваете должно использоваться, гибкость тут наоборот страдает.

[jstalex]

Лимиты ограничивают пользователя, для всех токенов они общие

А открытие счета добавляли с целью упростить запуск примеров - указать токен и все

[falkolab]

То есть для торговли на нескольких счетах, нужно запускать несколько приложений. На наш взгляд это позволяет более гибко управлять счетами и доступами к ним у разных ботов.

Лимиты ограничивают пользователя, для всех токенов они общие

Так каким образом вам получается гибко управлять счетами и доступами к ним с разных ботов, если в базовом грейде доступно только два потока на свечи, а с учетом того, как в API криво организована подписка особенно на свечи на открытые минутки, то и вовсе два потока уйдет на один бот. Ладно пошутили хватит, нет гибкости. Но ближе к делу.

А открытие счета добавляли с целью упростить запуск примеров - указать токен и все

А что мешает сделать проверку в примерах, создать песочницу перед созданием клиента и при этом сохранить чистоту кода SDK ?