-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add ydb_types-credentials-login to Helpers component #349
base: main
Are you sure you want to change the base?
Add ydb_types-credentials-login to Helpers component #349
Conversation
В Helpers добавлять не надо, предназначение этой компоненты в том, чтобы инициализировать CredentialsProvider на основе переменных окружения. Пока нет переменных для static аутентификации, значит и нет смысла добавлять туда. На мой взгляд, лучше пока сделать отдельный компонент Credentials, где будут все виды |
This reverts commit 532535d.
Спасибо за совет, сейчас реализую следуя ему. |
Сейчас все CredentialsProvider (за исключением Login) делятся по двум компонентам: Iam и Helpers. |
Я скорее думал добавить компоненту Credentials, которая будет содержать:
Это чтобы можно было прилинковать сразу все провайдеры кроме Iam (он особенный, ходит в сторонний IAM сервис, хочется дать возможность пользователю полностью выпилить его, чтобы не пугать) |
_ydb_sdk_make_client_component создает алиас для указанного таргета. Либо моих знаний cmake не хватает, и я не понимаю, как создать виртуальный таргет, для которого можно будет задать зависимости линковки и алиас. |
@Gazizonoki, я понял чего Вы от меня хотели) А слона то я и не заметил, у нас уже есть библиотека client-ydb_types-credentials, |
На текущий момент client-ydb_types-credentials-login не включен ни в один из компонентов, что усложняет использование, например в userver.
Добавил его в Helpers.