Tinkoff / investAPI

396 stars 136 forks source link

Вопрос по структуре common.proto #51

Closed SinnerUfa closed 2 years ago

SinnerUfa commented 2 years ago

Здравствуйте, возник небольшой вопрос по текущей структуре common.proto, а именно по:

//Денежная сумма в определенной валюте
message MoneyValue {
  // строковый ISO-код валюты
  string currency = 1;
  // целая часть суммы, может быть отрицательным числом
  int64 units = 2;
  // дробная часть суммы, может быть отрицательным числом
  int32 nano = 3;
}

//Котировка - денежная сумма без указания валюты
message Quotation {
  // целая часть суммы, может быть отрицательным числом
  int64 units = 1;
  // дробная часть суммы, может быть отрицательным числом
  int32 nano = 2;
}

Почему данную структуру сообщения не привести к виду:

//Денежная сумма в определенной валюте
message MoneyValue {
  // строковый ISO-код валюты
  string currency = 1;
reserved 2, 3
reserved  "units", "nano "
//Котировка 
Quotation value =4; // или 2 без резерва, пока версия в стадии beta замена номера некритична
}

//Котировка - денежная сумма без указания валюты
message Quotation {
  // целая часть суммы, может быть отрицательным числом
  int64 units = 1;
  // дробная часть суммы, может быть отрицательным числом
  int32 nano = 2;
}

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

Некоторые несущественные замечания:

  1. В документации https://tinkoff.github.io/investAPI/grpc/ довольно скудно описано про использование TLS/SSL в gRPC соединении, что приводит к некоторой неопределенности. Есть лишь упоминание в настройке Kreya и BloomRPC (хотя Kreya вообще эту настройку игнорирует) . Примечения:
  1. После ввода readonly токенов можно было бы указывать права доступа к конкретному счету (например через Accounts). А также характеристики самих токенов такие как время жизни токена.

Заранее спасибо за ответ.

SinnerUfa commented 2 years ago

Видимо неактуально)